⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
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
@aldbr aldbr linked an issue Feb 4, 2026 that may be closed by this pull request
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.

Stompy errors in matcher

3 participants