Skip to content

Add Feature-Policy header support #5156

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
Merged
traefiker merged 3 commits into from
Jul 29, 2019
Merged

Add Feature-Policy header support #5156

traefiker merged 3 commits into from
Jul 29, 2019

Conversation

@dtomcej
Copy link
Contributor

@dtomcej dtomcej commented Jul 24, 2019

What does this PR do?

Updates the unrolled/secure dependency, and implements the Feature-Policy header.

Motivation

Fixes #4936

More

  • Added/updated tests
  • Added/updated documentation

@traefiker traefiker added this to the 2.0 milestone Jul 24, 2019
SantoDE
SantoDE approved these changes Jul 26, 2019
View reviewed changes
Copy link
Collaborator

@SantoDE SantoDE left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Great :)

dduportal
dduportal approved these changes Jul 26, 2019
View reviewed changes
Copy link
Contributor

@dduportal dduportal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

jbdoumenjou
jbdoumenjou approved these changes Jul 29, 2019
View reviewed changes
Copy link
Collaborator

@jbdoumenjou jbdoumenjou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@traefiker traefiker merged commit cd164de into traefik:v2.0 Jul 29, 2019
@traefiker traefiker mentioned this pull request Jul 29, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jbdoumenjou jbdoumenjou jbdoumenjou approved these changes

@SantoDE SantoDE SantoDE approved these changes

+1 more reviewer

@dduportal dduportal dduportal approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area/middleware area/provider kind/enhancement a new or improved feature. size/M

Projects

None yet

Milestone

2.0

Development

Successfully merging this pull request may close these issues.

6 participants

@dtomcej @jbdoumenjou @SantoDE @dduportal @ldez @traefiker