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

Redirection status codes for methods different than GET #4116

Merged
merged 2 commits into from
Jan 7, 2019
Merged

Redirection status codes for methods different than GET #4116

merged 2 commits into from
Jan 7, 2019

Conversation

r--w
Copy link
Contributor

@r--w r--w commented Oct 26, 2018

For HTTP methods different than GET on redirection status codes should be:

#4040 #hacktoberfest #greywizard

dduportal
dduportal suggested changes Nov 6, 2018
View reviewed changes
docs/configuration/ping.md Outdated Show resolved Hide resolved
@ldez ldez added this to the 1.7 milestone Nov 7, 2018
@ldez ldez removed the bot/no-merge label Nov 7, 2018
@r--w r--w requested a review from a team as a code owner November 7, 2018 15:24
@mmatur mmatur changed the base branch from master to v1.7 November 7, 2018 15:24
@mmatur mmatur removed the request for review from a team November 7, 2018 15:24
dduportal
dduportal approved these changes Nov 11, 2018
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 thanks for this contribution!

SantoDE
SantoDE approved these changes Nov 12, 2018
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 👼Thanks

ldez
ldez approved these changes Jan 7, 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 b5a4d07 into traefik:v1.7 Jan 7, 2019
@ldez ldez mentioned this pull request Jan 8, 2019
Merged
16 tasks
@ldez ldez mentioned this pull request Mar 18, 2020
Closed
@rtribotte rtribotte mentioned this pull request Nov 14, 2024
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ldez ldez ldez approved these changes

@dduportal dduportal dduportal approved these changes

@SantoDE SantoDE SantoDE approved these changes

Assignees
No one assigned
Labels
area/middleware kind/bug/fix a bug fix size/S
Projects
None yet
Milestone
1.7
Development

Successfully merging this pull request may close these issues.
6 participants
@r--w @SantoDE @dduportal @ldez @juliens @traefiker