Hi @IMurzich,

Congratulations on submitting your first PR for the Spring Framework! 👍

However, before we process this PR, we need to take care of the following.

1. Please explain the error scenario you are attempting to fix.
2. Please introduce a unit or integration test which fails before the fix and passes after the fix.

Regarding point # 1, I realize you added some text to this issue's description, but it is not fully clear to me what you mean. So, perhaps you can just expound on that.

Thanks

Hi @sbrannen,

I've added test. Please check it.

Thanks.

Purpose of change:

I have a controller and exception handler in it.

My REST service can return JSON or XML replies.

Therefore I want to return exceptions in JSON or XML too.

Exception handler is annotated with:

@ExceptionHandler(exception = Exception.class, produces = {"application/json", "application/xml"})

And I expect to get JSON response with error (accept=*/*), but I receive XML response because HashSet doesn't retain the order of the produces media types and returns application/xml as the first media type.

@IMurzich, can you please sign off your commits so that the Developer Certificate of Origin (DCO) check passes?

Thanks

@IMurzich please sign the DCO. You can do that with by issuing the following commands on your fork:

git amend -s
git push --force

Fixed via 7699b4a

This has been merged into 6.2.x and main in 7699b4a and slightly revised in 8456cd1.

Thanks for your contribution! 👍