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.

Sign up for GitHub

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

Jump to bottom

Feature timeout in transport #78

Closed
wants to merge 4 commits into from
Closed

Feature timeout in transport #78

wants to merge 4 commits into from

Conversation

pchauhan-qlogic
Copy link

@pchauhan-qlogic pchauhan-qlogic commented Jul 25, 2019 •
edited by crwilcox
Loading

fixes #45 and #7

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 25, 2019
@crwilcox crwilcox added do not merge Indicates a pull request not ready for merge, due to either quality or timing. needs more info This issue needs more information from the customer to proceed. labels Jul 25, 2019
@crwilcox
Copy link
Collaborator

We are actively working on a design for retry/timeout but not ready to accept PRs for this at this time.

@crwilcox
Copy link
Collaborator

@pchauhan-qlogic while we closed this, it may be useful to make some notes. Can you explain the TimeoutAuthorizedSession and its purpose. The rest of this PR is mostly about plumbing kwargs, but is there something more there?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cla: yes This human has signed the Contributor License Agreement. do not merge Indicates a pull request not ready for merge, due to either quality or timing. needs more info This issue needs more information from the customer to proceed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add request timeout
3 participants
@pchauhan-qlogic @crwilcox @googlebot