Skip to content

Internal Timeout in GlobusExchangeTransport#394

Merged
AK2000 merged 4 commits into
mainfrom
ak2000-globus-exchange-recv-timeout
Apr 16, 2026
Merged

Internal Timeout in GlobusExchangeTransport#394
AK2000 merged 4 commits into
mainfrom
ak2000-globus-exchange-recv-timeout

Conversation

@AK2000

@AK2000 AK2000 commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Previously, the GlobusExchangeTransport relied on waiting on receive indefinitely for a message. This causes problems when connections timeout (i.e. because of site of nginx policies). The ideal solution would be to move to SSE (#393) but this is blocked a bug within the globus_sdk. This PR is a temporary patch so that we can move forward with enforcing a maximum timeout in the cloud exchange (#390). This PR essentially allows the GlobusExchangeTransport to deal with timeouts gracefully.

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 added the bug Something isn't working label Apr 16, 2026
@AK2000 AK2000 force-pushed the ak2000-globus-exchange-recv-timeout branch 2 times, most recently from cbc4ef3 to 8661202 Compare April 16, 2026 18:15
@AK2000 AK2000 merged commit 88ff988 into main Apr 16, 2026
13 checks passed
@AK2000 AK2000 deleted the ak2000-globus-exchange-recv-timeout branch April 16, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Globus Exchange Client Causes Timeouts

1 participant