Skip to content

Content-app doesn't respect response-headers specified by plugins #4028

Description

@ggainey

Version
3.28

Describe the bug
core/3.28 added Distribution.content_headers_for() to allow plugins to specifiy response-headers. When using non-FileStorage backends, these headers are ignored.

pulp_rpm PR pulp/pulp_rpm#3193 , for example, fails on azure and S3 because the cache-disposition header isn't ending up in the response.

To Reproduce

Expected behavior
headers added by a plugin should be made available in the response

Additional context
With azure and google, looks like we can just feed the headers into response-parameters. S3 may need specific code.

Metadata

Metadata

Assignees

Labels

Type

No type
No fields configured for issues without a type.

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions