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 extractorfunc values #4351

Merged
merged 1 commit into from
Jan 3, 2019
Merged

Add extractorfunc values #4351

merged 1 commit into from
Jan 3, 2019

Conversation

hsmade
Copy link
Contributor

@hsmade hsmade commented Jan 2, 2019 •
edited by ldez
Loading

As I had to look through code to find this. If it's in the docs somewhere and I missed it, please disregard

What does this PR do?

Update to docs

Motivation

Looking for the answer for too long ;o)

More

  • Added/updated documentation

Additional Notes

@hsmade
Proposing a small update to documentation
cc59094 
As I had to look through code to find this. If it's in the docs somewhere and I missed it, please disregard
@traefiker traefiker added this to the 1.7 milestone Jan 2, 2019
mmatur
mmatur approved these changes Jan 2, 2019
View reviewed changes
Copy link
Member

@mmatur mmatur left a comment

Choose a reason for hiding this comment

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

LGTM

dtomcej
dtomcej approved these changes Jan 3, 2019
View reviewed changes
Copy link
Contributor

@dtomcej dtomcej left a comment

Choose a reason for hiding this comment

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

LGTM
:shipit:

ldez
ldez approved these changes Jan 3, 2019
View reviewed changes
Copy link
Contributor

@ldez ldez 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 0e643f0 into traefik:v1.7 Jan 3, 2019
@ldez ldez changed the title Proposing a small update to documentation Add extractorfunc values Jan 3, 2019
@ldez ldez mentioned this pull request Jan 8, 2019
Merged
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmatur mmatur mmatur approved these changes

@ldez ldez ldez approved these changes

@dtomcej dtomcej dtomcej approved these changes

Assignees
No one assigned
Labels
area/documentation area/middleware kind/enhancement a new or improved feature. size/S
Projects
None yet
Milestone
1.7
Development

Successfully merging this pull request may close these issues.
6 participants
@hsmade @mmatur @ldez @dtomcej @geraldcroes @traefiker