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

Add Content-Length header to file download REST API #1212

Closed
luis100 opened this issue Apr 27, 2018 · 1 comment
Closed

Add Content-Length header to file download REST API #1212

luis100 opened this issue Apr 27, 2018 · 1 comment
Assignees
@nunovieira220
Labels
enhancement
Milestone
3.0

Comments

@luis100
Copy link
Member

luis100 commented Apr 27, 2018

On API request:
/api/v1/files/FILEID?acceptFormat=bin
The Content-Length header is not set.

Add file size (if the information is already available) to the StreamResponse and improve ApiUtils to include the Content-Length header.

This is necessary to improve the file previewers.
@luis100 luis100 added this to the 2.3.0 milestone Apr 27, 2018
@luis100 luis100 self-assigned this Apr 27, 2018
@luis100 luis100 assigned nunovieira220 and unassigned luis100 May 24, 2018
nunovieira220 added a commit that referenced this issue May 28, 2018
@nunovieira220
added content length header when downloading files via API #1212
74728ec
@nunovieira220
Copy link
Contributor

Done

luis100 pushed a commit that referenced this issue May 30, 2018
@nunovieira220 @luis100
added content length header when downloading files via API #1212
97cdd79
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nunovieira220 nunovieira220

Labels
enhancement
Projects
None yet
Milestone
3.0
Development

No branches or pull requests
2 participants
@luis100 @nunovieira220