Feature - temporary urls

[xavierlacot]

fix #75
Co-authored with @rvanlaak, supersedes #72 and #79

Add native temporary (pre-signed) URL generation for medias and their variations, so applications backed by an S3-compatible bucket can serve files directly instead of proxying them through PHP.

A new getTemporaryUrl() method is available on Media, MediaVariation and both storages, signing the actual storage path with a configurable expiration.

The bundle also stops forcing a public visibility when writing files, so the visibility configured on the Flysystem storage now applies, which is required for private buckets.

[rvanlaak]

This is great! I'll try verifying it asap, and whether my custom path replacement normalizer can be removed when this PR ships.

[damienalexandre]

@rvanlaak Hello! 👋 Have you been able to test the feature?

[rvanlaak]

Not yet. Can verify it Tomorrow.

[xavierlacot]

Hi @rvanlaak, I rebased this PR on main and added a few things on top.

Variation existence is now cached once a file is known to be stored, so signing the URL of an existing variation no longer sends a HEAD request to the bucket on every call, which should address the slowness you noticed on your GET endpoints.

Also:

• NullMedia::getTemporaryUrl() now throws like getUrl() does
• the signed path ignores url_generator.path, so the normalizer rewriting your paths should no longer be needed
• Temporary URLs are also covered by a new test suite running against an S3-compatible server

Except if you have a negative feedback on this PR, I am going to merge it by the end of the week.

[rvanlaak]

Marked a block in my calendar for tomorrow afternoon. If I can't make it to provide feedback by eod I will also let you know.

[rvanlaak]

Hi @xavierlacot , I've checked out and tested locally, and can confirm that all works as expected. It will allow me to remove little pieces of my custom code, especially the part related to what you're mentioning about the HEAD calls to the S3 buckets.

A big part of it is related to my custom normalizers, but that code is unrelated to this PR and I'll try drafting another PR for this repo to get that part rolling too.

[xavierlacot]

Alright then, thanks for the feedback and your contributions :-)