Provided brotli support. by Rohitub222 · Pull Request #317 · appwrite/appwrite · GitHub
Skip to content

Provided brotli support. - #317

Closed
Rohitub222 wants to merge 2 commits into
appwrite:masterfrom
Rohitub222:feature-brotli-support
Closed

Rohitub222 wants to merge 2 commits into
appwrite:masterfrom
Rohitub222:feature-brotli-support

Conversation

@Rohitub222

Copy link
Copy Markdown

No description provided.

@eldadfux

Copy link
Copy Markdown
Member

@Rohitub222

Copy link
Copy Markdown
Author

Hi @eldadfux I have updated the brotli installation using apt-get ( thanks for the doc ).

@eldadfux

Copy link
Copy Markdown
Member

Hey @Rohitub222 build is still failing, I think it will be easier for you to test this locally.

@W4nn4Die

W4nn4Die commented May 28, 2020 •

Copy link
Copy Markdown
Contributor

Hello @Rohitub222 seems like there's an error with the nginx module brotli, take a look at this url, hope it helps you 😉

@eldadfux

Copy link
Copy Markdown
Member

@PedroCisnerosSantana We were thinking about installing it from the source of the official repo:
https://github.com/google/brotli

@W4nn4Die

Copy link
Copy Markdown
Contributor

@eldadfux why don't we just use a brotli docker? check this

@eldadfux

Copy link
Copy Markdown
Member

@PedroCisnerosSantana actually that can be a very good reference... what do you say @Rohitub222 ?

@Rohitub222

Rohitub222 commented May 29, 2020 via email

Copy link
Copy Markdown
Author

@eldadfux

Copy link
Copy Markdown
Member

Hey @Rohitub222, any updates?

@eldadfux

Copy link
Copy Markdown
Member

@eldadfux eldadfux closed this Jun 13, 2020
pull Bot pushed a commit to TheTechOddBug/appwrite that referenced this pull request Sep 28, 2026
refactor(queue): use publishMany for synchronous batches
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