WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@chrisburr
Copy link
Member

BEGINRELEASENOTES

*Core
FIX: Add wait timeout when connecting to stomp

ENDRELEASENOTES

@chrisburr chrisburr requested a review from chaen February 24, 2025 09:09
@DIRACGridBot DIRACGridBot added the alsoTargeting:integration Cherry pick this PR to integration after merge label Feb 24, 2025
self.connection.connect(username=user, passcode=password, wait=True)

# Connect to the broker with a timeout of 30 seconds
self.connection.connect(username=user, passcode=password, wait=False)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It isn't as wait=True blocks forever and doesn't have a timeout.

@chrisburr chrisburr force-pushed the stomp-connect-timeout branch from c4a0036 to 29b8993 Compare May 20, 2025 09:53
@chrisburr chrisburr force-pushed the stomp-connect-timeout branch from 29b8993 to 1c36a20 Compare May 20, 2025 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

alsoTargeting:integration Cherry pick this PR to integration after merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants