Skip to content

Add brotli extension support#547

Open
otezz wants to merge 1 commit into
brefphp:masterfrom
otezz:ext-brotli
Open

Add brotli extension support#547
otezz wants to merge 1 commit into
brefphp:masterfrom
otezz:ext-brotli

Conversation

@otezz

@otezz otezz commented Jul 12, 2024

Copy link
Copy Markdown

Add Brotli support.

Since neither AWS HTTP API Gateway nor CloudFront has the ability to compress responses with Brotli, we can achieve this by using something like https://github.com/open-southeners/laravel-vapor-response-compression

@GrahamCampbell

GrahamCampbell commented Jul 12, 2024

Copy link
Copy Markdown
Contributor

I think this was an intentional decision by AWS since the compute requirements for compression are very high, an order of magnitude more than for gzip, and the average compression ratio is barely better.

@otezz

otezz commented Jul 15, 2024

Copy link
Copy Markdown
Author

They support payload compression on REST API but not in HTTP API (yet). I think it should be okay to give users the option to handle the compression themselves.

@Nyholm Nyholm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I understand that this dont always makes sense. But I am happy to merge this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants