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

Disable timeout by default, add timeout= option #10

Closed
simonw opened this issue Nov 6, 2024 · 1 comment
Closed

Disable timeout by default, add timeout= option #10

simonw opened this issue Nov 6, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@simonw
Copy link
Owner

simonw commented Nov 6, 2024

I was trying to use this to proxy OpenAI but there's a default timeout of 5s which broke it.
@simonw simonw added the enhancement New feature or request label Nov 6, 2024
@simonw
Copy link
Owner Author

simonw commented Nov 6, 2024

If I turn timeout into an option, what should happen when it is exceeded? It should be a 504 error.

@simonw simonw closed this as completed in 96162d7 Nov 6, 2024
simonw added a commit that referenced this issue Nov 6, 2024
@simonw
Release 0.2a0
098d767 
Refs #9, #10
simonw added a commit that referenced this issue Nov 6, 2024
@simonw
Fix tests, add test for timeout, refs #10
edf556a
@simonw simonw changed the title Disable timeout by default Disable timeout by default, add timeout= option Nov 6, 2024
simonw added a commit that referenced this issue Nov 6, 2024
@simonw
Drop Python 3.9 as well
2417aff 
It's EOL as of last month.

Hopefully will get tests passing for #10
@simonw simonw mentioned this issue Nov 6, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@simonw