supabase / supabase-flutter / 24993319290

80%

push

github

fix(storage): avoid duplicate Content-Type headers and header mutation (#1359)

* fix(storage): avoid duplicate Content-Type headers and header mutation

In _handleRequest, the headers map was being mutated directly by adding
Content-Type: application/json. This meant:

1. The stored headers map on StorageFileApi/_StorageBucketApi was permanently
   mutated after each non-GET request, accumulating Content-Type.
2. Custom Content-Type headers set via setHeader() were overwritten
   unconditionally, ignoring caller intent.

Fix: copy headers before modifying, and use a case-insensitive check
for an existing Content-Type before setting the application/json default.

Linear: SDK-881

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* style(storage): format fetch.dart

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* Update packages/storage_client/lib/src/fetch.dart

Co-authored-by: Vinzent <vinzent03@proton.me>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: Vinzent <vinzent03@proton.me>

3 of 3 new or added lines in 1 file covered. (100.0%)

3529 of 4389 relevant lines covered (80.41%)

2.86 hits per line