Making Uniform bucket-level access error more visible.

[jamessa]

As I can setup download in 5 mins, upload got me 3 hours for two reasons,

1. My devOps didn't tell me anything about permission, so
2. I overlooked https://github.com/spatie/laravel-google-cloud-storage?tab=readme-ov-file#uniform-bucket-level-access completely.

I trace into Google cloud storage and finally found this.

{
  "error": {
    "code": 400,
    "message": "Cannot insert legacy ACL for an object when uniform bucket-level access is enabled. Read more at https://cloud.google.com/storage/docs/uniform-bucket-level-access",
    "errors": [
      {
        "message": "Cannot insert legacy ACL for an object when uniform bucket-level access is enabled. Read more at https://cloud.google.com/storage/docs/uniform-bucket-level-access",
        "domain": "global",
        "reason": "invalid"
      }
    ]
  }
}

Finally I found throw option in FilesystemAdapter.php

/**
 * Determine if Flysystem exceptions should be thrown.
 *
 * @return bool
 */
protected function throwsExceptions(): bool
{
    return (bool) ($this->config['throw'] ?? false);
}

https://github.com/laravel/framework/blob/07e12d378ec0875cdcfe4272d120d56e2312f713/src/Illuminate/Filesystem/FilesystemAdapter.php#L967

So I went back to config/filesystems.php

'disks' => [
    'gcs' => [
        'driver' => 'gcs',
        'throw' => true,
        // skip ...
    ]
]

Now I can see this exception right from my test.

test('Google Cloud Test', function () {
    $prefix = 'tests/';

    $disk = Storage::disk('gcs');

    $disk->put($prefix.'hi.txt', 'Hello World');
    expect($disk->get($prefix.'hi.txt'))->toBe('Hello World');

    $disk->delete($prefix.'hi.txt');

})->onlyOnMac(); // 只在 local 環境執行

Unable to write file at location: tests/hi.txt. {
  "error": {
    "code": 400,
    "message": "Cannot insert legacy ACL for an object when uniform bucket-level access is enabled. Read more at https://cloud.google.com/storage/docs/uniform-bucket-level-access",
    "errors": [
      {
        "message": "Cannot insert legacy ACL for an object when uniform bucket-level access is enabled. Read more at https://cloud.google.com/storage/docs/uniform-bucket-level-access",
        "domain": "global",
        "reason": "invalid"
      }
    ]
  }
}

Conclusion

Add 'throw' in default gcs configuration for easy debugging.