Allow customizing RPC request method

[bnjmnt4n]

What kind of change does this PR introduce?

Feature.

What is the current behavior?

RPC function calls are made only using POST.

What is the new behavior?

Users can customize the HTTP request type to GET or HEAD. This allows for selecting the count of rows in a function call returning a table without selecting any data with the HEAD request type and count option.

Implementation Notes

• I made use of Object.entries, but if there's any browser compatibility requirements I can just revert to a for-in loop.
  • This also required editing the tsconfig.json to recognize newer ES methods
• I also didn't account for the edge case where an array parameter is passed with a GET/HEAD request. Seems like there are two possible usages of this: https://postgrest.org/en/v8.0/api.html#calling-functions-with-array-parameters and https://postgrest.org/en/v8.0/api.html#calling-variadic-functions and I'm not sure behaviour which to prefer.

[soedirgo]

Thanks for the PR! Somehow I forgot to support HEAD for rpc.

For HEAD, I think it's better to follow how it's done for select instead:

postgrest-js/src/lib/PostgrestQueryBuilder.ts

Lines 22 to 31 in 5e332bf

	select(
	columns = '*',
	{
	head = false,
	count = null,
	}: {
	head?: boolean
	count?: null | 'exact' | 'planned' | 'estimated'
	} = {}
	): PostgrestFilterBuilder<T> {

Looking at the docs, it doesn't seem like there's anything GET does that POST can't do, and sending params as request body is cleaner (URL has length limit and all that), so supporting GET probably isn't necessary.

[bnjmnt4n]

One case I could think of (although certainly niche) is that service workers are unable to cache POST requests.

[soedirgo]

Hmm yeah, I don't have anything against it but maybe best to leave it out until there's a feature request for it.

[bnjmnt4n]

@soedirgo updated! I guess it's good to avoid unnecessary complexity until it's really required, and this does have more consistency with select.

[bnjmnt4n]

Should I send a corresponding PR to integrate this into supabase-js?

[github-actions]

🎉 This PR is included in version 0.34.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

[soedirgo]

Yes please, that'd be rad 🙏