Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

3rd party API: enable compression #50

Closed
korelstar opened this issue Jan 18, 2017 · 2 comments
Closed

3rd party API: enable compression #50

korelstar opened this issue Jan 18, 2017 · 2 comments
Labels
enhancement New feature or request feature: API Related to the API for 3rd-party apps, i.e. Android or iOS

Comments

@korelstar
Copy link
Member

korelstar commented Jan 18, 2017 •
edited by jospoortvliet
Loading

In order to speed up synchronization with mobile devices (mobile connections often are slow and unreliable) it would be advantageous if the connection would be automatically compressed. Depending on the value of the HTTP header Accept-Encoding sent by the client (Android is sending this automatically), the response should be compressed and the respective header Content-Encoding should be activated.

As far as I can see, the ownCloud/Nextcloud App Framework does not provide any support for this. Is this really true? If yes, then we would have to implement it on our own. This is not a big deal, but requires to pick the Accept-Encoding into pieces. As an alternative one could try to use features from the web server. However, for Apache it wouldn't be straight forward, because a .htaccess file in this app would not apply to requests to the API (they go over index.php from nextcloud/server). Therefore, I'm not sure which solution would be best. Any ideas?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.
@korelstar korelstar added enhancement New feature or request feature: API Related to the API for 3rd-party apps, i.e. Android or iOS labels Jan 18, 2017
@jancborchardt
Copy link
Member

Any idea about this @nextcloud/notes? Or @MorrisJobke @rullzer?

@korelstar
Copy link
Member Author

Will be fixed by Nextcloud 20, see nextcloud/server#20270

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request feature: API Related to the API for 3rd-party apps, i.e. Android or iOS
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jancborchardt @korelstar