fix(storage): local storage presigned urls

[avallete]

What kind of change does this PR introduce?

• The kong transform shoudn't be applied over S3 storage endpoint so the right token can be passed along.

Tests

Bisected to the right commit with this script:

https://github.com/supabase/cli/blob/85a4bf736bb69a1936dcec70e1572ef8b7c0cdb3/test.ts

And running:

go run . stop --all --no-backup && go run . start && deno run --allow-env --allow-net --allow-read --allow-sys test.ts

To check when the bug was introduced. Found the culprit here: ac5c902

We now bypass the kong transformation when the url is an url of the S3 protocol.

Improvements:

Might wanna setup some e2e tests on this so we don't experience regression like this in the future.

[coveralls]

Pull Request Test Coverage Report for Build 18661732577

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 5 unchanged lines in 1 file lost coverage.
• Overall coverage decreased (-0.03%) to 54.579%

---

Files with Coverage Reduction	New Missed Lines	%
internal/gen/keys/keys.go	5	12.9%

Totals
Change from base Build 18649941002:	-0.03%
Covered Lines:	6389
Relevant Lines:	11706

---

💛 - Coveralls

[sweatybridge]

good catch