Skip to content

fix(functions): set request timeout to 150 seconds when invoking functions #728

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jun 10, 2025

Conversation

grdsdev
Copy link
Collaborator

@grdsdev grdsdev commented Jun 5, 2025

What kind of change does this PR introduce?

Bug fix
Close #727

What is the current behavior?

No request timeout defined, which uses default URLSession timeout which is 60 seconds.

What is the new behavior?

Allow to specify a custom value for timeout a per-request basis, (internally, not configurable by the user)
Set all invoke functions requests timeout to 150 seconds

Additional context

https://supabase.com/docs/guides/functions/limits

@coveralls
Copy link

Pull Request Test Coverage Report for Build 15469746645

Details

  • 5 of 6 (83.33%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall first build on guilherme/clibs-190-edge-function-timeout-60-seconds at 75.659%

Changes Missing Coverage Covered Lines Changed/Added Lines %
Sources/Helpers/HTTP/HTTPRequest.swift 3 4 75.0%
Totals Coverage Status
Change from base Build 15390678797: 75.7%
Covered Lines: 5225
Relevant Lines: 6906

💛 - Coveralls

@grdsdev grdsdev marked this pull request as ready for review June 9, 2025 09:55
@grdsdev grdsdev requested a review from dshukertjr June 9, 2025 09:55
@grdsdev
Copy link
Collaborator Author

grdsdev commented Jun 9, 2025

@dshukertjr not sure if this happens in Flutter too, does HTTPClient has a default timeout for the requests? If that is the case, this will need to be fixed on flutter too.

@grdsdev grdsdev merged commit 3d2930d into main Jun 10, 2025
25 checks passed
@grdsdev grdsdev deleted the guilherme/clibs-190-edge-function-timeout-60-seconds branch June 10, 2025 09:42
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Edge function timeout 60 seconds
3 participants