Check hasNext when when accessing sessionIds from UserDestinationResult - #34333

Found a bug in UserDestinationMessageHandler.SendHelper#send that occurs when the UserDestinationResult was created by the pre-6.1.x constructor. Bug was introduced with 3277b0d

Ideally this fix will target 6.1+ and should be compatible. Not sure if there's a special way to call that out as a first-time contributor.

Changes

• Add a test that reproduces the error and subsequently covers the fix
• Check if the iterator has a next value before accessing it and use the default condition if it does not
  • Using the default seems to be the original intent and was just missing this initial check

Bug Details

The UserDestinationMessageHandler unsafely retrieves values from an iterator of sessionId strings to lookup ordered messages.

This happens when the UserDestinationResult used to route the message was created with the pre-6.1.x constructor this retrieval throws a NoSuchElementException.

This is because the older constructor passes a null to the newer constructor for the sessionIds constructor arg. The class attempts to safe handle this null arg by creating an EmptySet that eventually provides an [EmptyIterator],(https://github.com/openjdk/jdk/blob/98a93e115137a305aed6b7dbf1d4a7d5906fe77c/src/java.base/share/classes/java/util/Collections.java#L4559) which then throws for Set#next.

This bug seems to only happen for custom implementations of UserDestinationResolver as the DefaultUserDestinationResolver always provides a populated set to the new constructor.