Skip to content
This repository was archived by the owner on Oct 9, 2025. It is now read-only.

Conversation

dsc8x
Copy link
Contributor

@dsc8x dsc8x commented Dec 30, 2024

What kind of change does this PR introduce?

Adds support for passing an AbortSignal to function invocations

What is the current behavior?

Functions cannot be aborted. This is bad if you for example use an edge function to fetch input autocompletion results, as you ideally want to cancel all outdated requests if the input changes.

Please link any relevant issues here.

https://github.com/orgs/supabase/discussions/27660

What is the new behavior?

Functions can be cancelled by adding the signal option

@jens-f
Copy link

jens-f commented Jan 19, 2025

We would also need this same exact functionality. Would it be possible to get this PR reviewed and merged?

@ugoi
Copy link

ugoi commented Apr 12, 2025

Any updates on this?

Copy link

@ugoi ugoi left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link

🚀 Preview Release Status

false


Last updated: 2025-09-16T00:49:47Z

@laktek laktek merged commit dc5eee7 into supabase:main Sep 16, 2025
3 checks passed
@coveralls
Copy link

Pull Request Test Coverage Report for Build 17750882289

Details

  • 10 of 14 (71.43%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-1.7%) to 90.657%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/FunctionsClient.ts 6 10 60.0%
Totals Coverage Status
Change from base Build 17750819185: -1.7%
Covered Lines: 230
Relevant Lines: 247

💛 - Coveralls

@laktek
Copy link
Contributor

laktek commented Sep 16, 2025

Thanks for the PR @dsc8x! Sorry for the delay in merging it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants