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.

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

fix(api): stream api response to function to avoid timeout #2760

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Maby0
Copy link
Collaborator

@Maby0 Maby0 commented Dec 16, 2024

Describe your changes

This is a change that was made as a PoC for streaming responses back to avoid HTTP timeouts on Azure. Separate PRs will be made to implement any required changes.

Issue ticket number and link

Type of change 🧩

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Other (please explain in the description section above)

Checklist before requesting a review

  • I have performed a self-review of my own code
  • I have double checked this work does not include any hardcoded secrets or passwords
  • I have made corresponding changes to the documentation
  • I have provided details on how I have tested my code
  • I have referenced the ticket number above
  • I have provided a description of the ticket
  • I have included unit tests to cover any testable code changes

@Maby0 Maby0 force-pushed the fix/applics-1114-stream-migration-response branch 2 times, most recently from 8707be0 to 7297970 Compare December 17, 2024 13:39
@Maby0 Maby0 force-pushed the fix/applics-1114-stream-migration-response branch from 0e5039a to f3d43b7 Compare January 3, 2025 14:20
@Maby0 Maby0 force-pushed the fix/applics-1114-stream-migration-response branch 2 times, most recently from 3e7b5fd to a03f4c5 Compare January 3, 2025 14:58
@Maby0 Maby0 force-pushed the fix/applics-1114-stream-migration-response branch from a03f4c5 to 48e06d4 Compare January 3, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant