Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[UNDERTOW-2338] Use a lock when setting the AsyncContext and asyncSta… #1546

Closed
wants to merge 1 commit into from

Conversation

jamezp
Copy link
Contributor

@jamezp jamezp commented Feb 6, 2024

…rted variables. There is a race condition on when the isAsyncStarted() may return true before the asyncContext is set.

https://issues.redhat.com/browse/UNDERTOW-2338

…rted variables. There is a race condition on when the isAsyncStarted() may return true before the asyncContext is set.

Signed-off-by: James R. Perkins <jperkins@redhat.com>
@fl4via fl4via added the duplicate Duplicates other pull request(s) label Feb 13, 2024
@fl4via
Copy link
Member

fl4via commented Feb 13, 2024

@jamezp as we discussed in Zulip, I investigated if I think it can be fixed without the lock and it can. I opened a new PR: #1546 and I am marking this one as duplicate for now. If the error persists in RestEasy CI, please let me know

@fl4via fl4via closed this Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate Duplicates other pull request(s)
Projects
None yet
2 participants