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

Deadlock while using SSL + Proxy #398

Closed
JosephP91 opened this issue Jun 1, 2021 · 20 comments
Closed

Deadlock while using SSL + Proxy #398

JosephP91 opened this issue Jun 1, 2021 · 20 comments
Labels

Comments

@JosephP91
Copy link

Hi all! I found a deadlock while looking at this thread dump. It seems to be related to SSL and proxy.

https://www.dropbox.com/s/4bpyevdcrbdgmya/thread-dump-clean.txt?dl=0

@JosephP91 JosephP91 added the bug label Jun 1, 2021
@chrjohn
Copy link
Member

chrjohn commented Jun 1, 2021 via email

@chrjohn
Copy link
Member

chrjohn commented Jun 1, 2021

Probably the best we can do is to open a MINA bug. But I saw there still is one related report open with them which sounds similar.

@JosephP91
Copy link
Author

Yes, are you referring to this? https://issues.apache.org/jira/browse/DIRMINA-1119

@chrjohn
Copy link
Member

chrjohn commented Jun 1, 2021 via email

@JosephP91
Copy link
Author

JosephP91 commented Jun 2, 2021

I openend a new one and attached the thread dump: https://issues.apache.org/jira/browse/DIRMINA-1144

I noticed that the deadlock does not occurr in the mina version 2.0.20, but I want to do more test just to be 100% sure.

@chrjohn
Copy link
Member

chrjohn commented Jun 2, 2021 via email

@JosephP91
Copy link
Author

I can confirm that the mina-core 2.0.20 version is working. So basically, if you exclude mina-core from quickfix dependency inside pom.xml file and include the 2.0.20 version, it should be fine. Since this is not a quickfix bug, I think I can close the issue. Feel free to reopen it for further problems.

@chrjohn
Copy link
Member

chrjohn commented Jun 4, 2021 via email

@swapnilgupta10
Copy link

@JosephP91 i have similar issue which i downgraded to 2.0.23 but still figuring dead lock issue. Wondering what led us to 2.0.20 to make sure it's working.? Do we have any release not for this issue to occur after any specific version?

@chrjohn
Copy link
Member

chrjohn commented Aug 8, 2023

This hopefully should be fixed with MINA 2.2.x
See #635

@swapnilgupta10
Copy link

@chrjohn Thanks for quick update.
As per your above pull request, look like we need mina version upgrade along with few changes in quick fix api.
Do we have any ETA to new release version for above change? Will this required jdk higher version than 1.8 +?

@chrjohn
Copy link
Member

chrjohn commented Aug 8, 2023

No, we do not have an ETA currently, still working on the 3.0.0 release which will most likely still use JDK8.
However, maybe we could also do a 2.3.x release with this change but actually I'd like that someone tests if that problem is really fixed.

@swapnilgupta10
Copy link

That would be great if we can cut a release 2.3.x with mina 2.2.2. I could perform some sanity testing if that's possible. (Will be difficult to reproduce this issue on our lower environment). Let me know when if that's possible. Thanks

Also, as per https://issues.apache.org/jira/browse/DIRMINA-1132
Look like with 2.2.2, entire SSL implementation has been changed. Also environment section under above ticket relates > JDK 1.8

@chrjohn
Copy link
Member

chrjohn commented Aug 8, 2023

MINA 2.2.2 is still JDK8, otherwise #635 could not have been built.

@chrjohn
Copy link
Member

chrjohn commented Aug 16, 2023

I have merged the changes into https://github.com/quickfix-j/quickfixj/tree/QFJ_2_3_x
Are you able to build a release from there and test it?

@swapnilgupta10
Copy link

Sure, I'll give a try. You mean build a snapshot release on my local and test?

@chrjohn
Copy link
Member

chrjohn commented Aug 16, 2023

Yes, exactly.

@x-leviathan
Copy link

Hi @chrjohn
Does recent released version 2.3.2 fix this deadlock bug? I notice quickfixj 2.3.2 uses Mina 2.2.4

@chrjohn
Copy link
Member

chrjohn commented Jan 21, 2025

#774 (reply in thread)

@x-leviathan
Copy link

Got it. Thanks. I will try to perform integration testing locally to verify.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants