Skip to content

Switch GlobusExchangeTransport to use SSE#393

Draft
AK2000 wants to merge 2 commits into
mainfrom
ak2000-globus-exchange-use-sse
Draft

Switch GlobusExchangeTransport to use SSE#393
AK2000 wants to merge 2 commits into
mainfrom
ak2000-globus-exchange-use-sse

Conversation

@AK2000

@AK2000 AK2000 commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

Summary

The GlobusExchangeTransport has fallen behind the HTTPExchangeTransport where we already switched to SSE events. This PR changes the GlobusExchangeTransport to also use SSE, which also allows us to more gracefully deal with timeouts and mailbox closed issues. This is immediately relevant to using the GlobusExchangeClient with persistent agents, where Nginx will raise Timeout errors that will break the server listening loop.

Related Issues

Changes

  • Breaking (backwards incompatible changes to public interfaces)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (non-breaking change or feature addition)
  • Refactor (internal code or design clean up)
  • Documentation (no changes to the code)
  • Test (changes or additions to testing)
  • Build (change to CI workflows or build processes)
  • Package (changes to package metadata or dependency versions)

Testing

N/A

Pull Request Checklist

Please confirm the PR meets the following requirements.

  • Relevant tags are added based on the types of changes.
  • Code changes pass pre-commit (e.g., ruff, mypy, etc.).
  • Tests have been added to show the fix is effective or that the new feature works.
  • New and existing unit tests pass locally with the changes.
  • Docs have been updated and reviewed if relevant.

@AK2000 AK2000 marked this pull request as draft April 15, 2026 21:32
@AK2000

AK2000 commented Apr 15, 2026

Copy link
Copy Markdown
Contributor Author

This is blocked by a bug in the globus sdk where streaming does not work as expected

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant