Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merge staging into main for Assurance 3.0.6 #156

Merged
merged 8 commits into from
Feb 25, 2025
Merged

Merge staging into main for Assurance 3.0.6 #156

merged 8 commits into from
Feb 25, 2025

Conversation

prudrabhat
Copy link
Contributor

@prudrabhat prudrabhat commented Feb 25, 2025

Description

Merge staging into main for Assurance 3.06

Fixed an issue that prevented a connection error message from being displayed.

Related Issue

prudrabhat and others added 8 commits January 21, 2025 11:47
Problem:

`notifyTerminationAndRemoveStatusListeners()` iterates over all the connection
status listeners to notify disconnection and un-registers them. Since the
de-registration process involves removing the listeners from the set, iterating over
the same list for removal leads to ConcurrentModificationException and not all listeners
are notified of the error. This results in the connecting icon spinning forever.

Solution:

Iterate for  de-registration over a copy of the listeners to prevent the exception
and propogate the error.
Reverse merge `main` into `dev-3.x`
MOB-22620: Fix concurrent modification exception during session termination
Merge `dev-v3.x` into `staging` for Assurance 3.0.6
@prudrabhat prudrabhat requested a review from praveek February 25, 2025 00:26
@prudrabhat prudrabhat changed the title Merge staging into main for Assurance 3.06 Merge staging into main for Assurance 3.0.6 Feb 25, 2025
@prudrabhat prudrabhat merged commit bfcd40b into main Feb 25, 2025
10 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants