Project

Profile

Help

Task #2914

closed

Task #2868: Platform support for publishing.

Add support for Pulp to handle X-SEND and X-Accel-Redirect response headers

Added by jortel@redhat.com over 7 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 28
Quarter:

Description

The ContentView needs to be updated to detect if XSEND can/should be used to stream bits. The pulp2 code[0] to add the X-SENDFILE header should also be ported.

It should be noted that nginx does not use the X-SENDFILE header, but rather the X-Accel-Redirect header.

It would be good to support either of these servers. Prehaps a default property named webserver in settings.py and a user config in server.yaml that defaults to None, and can be set to None, apache, or nginx

[0] https://github.com/pulp/pulp/blob/96db0a2eefe12683cd439d9a00e57a4da0876dbc/server/pulp/server/content/web/views.py#L74

Actions #1

Updated by jortel@redhat.com over 7 years ago

  • Description updated (diff)
Actions #2

Updated by bmbouter over 7 years ago

  • Subject changed from Add support for Apache integration and X-SEND files in Content WSGI application. to Add support for Apache integration and X-SEND files

I think for the foreseable future we will only have 1 WSGI application serving all content requests include API and content requests. I think the adding of XSend support and Apache can continue with that 1 WSGI application and if we want to break it out we could do that as separate work.

Actions #3

Updated by bizhang over 7 years ago

  • Subject changed from Add support for Apache integration and X-SEND files to Add support for Pulp to handle X-SEND files
  • Description updated (diff)

After discussion with @jortel I'm rewriting this to be a xsend task and am creating a new task to create an ansible apache role

Actions #4

Updated by bmbouter about 7 years ago

  • Subject changed from Add support for Pulp to handle X-SEND files to Add support for Pulp to handle X-SEND and X-Accel-Redirect response headers
  • Description updated (diff)

Retitling, added checklist items, and groomed. I also updated the description to specify the name webserver.

Actions #5

Updated by bmbouter about 7 years ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes
Actions #6

Updated by mhrivnak about 7 years ago

  • Sprint/Milestone set to 47
Actions #7

Updated by jortel@redhat.com about 7 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to jortel@redhat.com
Actions #8

Updated by jortel@redhat.com about 7 years ago

  • Status changed from ASSIGNED to POST

Added by jortel@redhat.com almost 7 years ago

Revision ec922115 | View on GitHub

Support x-send and nginx headers. closes #2914

Added by jortel@redhat.com almost 7 years ago

Revision ec922115 | View on GitHub

Support x-send and nginx headers. closes #2914

Actions #9

Updated by jortel@redhat.com almost 7 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #10

Updated by bmbouter over 6 years ago

  • Sprint set to Sprint 28
Actions #11

Updated by bmbouter over 6 years ago

  • Sprint/Milestone deleted (47)
Actions #12

Updated by daviddavis over 5 years ago

  • Sprint/Milestone set to 3.0.0
Actions #13

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions #14

Updated by bmbouter almost 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF