Skip to content

πŸ§‘β€πŸ’» Handle HTTP errors when streaming build logs#65

Merged
patrick91 merged 1 commit intomainfrom
07-25-_handle_http_errors_when_streaming_build_logs
Jul 28, 2025
Merged

πŸ§‘β€πŸ’» Handle HTTP errors when streaming build logs#65
patrick91 merged 1 commit intomainfrom
07-25-_handle_http_errors_when_streaming_build_logs

Conversation

@patrick91
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@patrick91 patrick91 added the feature New feature or request label Jul 25, 2025

super().__init__(
base_url=settings.base_api_url,
timeout=httpx.Timeout(20),
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Took the opportunity to update this too, mainly for staging, but default timeout was 5 seconds 😊

Copy link
Copy Markdown
Contributor

@tiangolo tiangolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! πŸš€

@patrick91 patrick91 merged commit 782d6b2 into main Jul 28, 2025
19 of 22 checks passed
@patrick91 patrick91 deleted the 07-25-_handle_http_errors_when_streaming_build_logs branch July 28, 2025 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants