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.

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

Continue processing on invalid auth-realm annotation. #2252

Conversation

timoreimann
Copy link
Contributor

We previously returned an error, thereby prematurely terminating any further Ingress processing.

Fixes #2239.

@timoreimann timoreimann requested a review from a team as a code owner October 11, 2017 20:29
@timoreimann timoreimann changed the base branch from master to v1.4 October 11, 2017 20:29
@ldez ldez added this to the 1.4 milestone Oct 11, 2017
Copy link
Contributor

@nmengin nmengin left a comment

Choose a reason for hiding this comment

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

LGTM 👏

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

Copy link
Member

@emilevauge emilevauge left a comment

Choose a reason for hiding this comment

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

Thanks @timoreimann
LGTM :)

We previously returned an error, thereby prematurely terminating any
further Ingress processing.
@traefiker traefiker force-pushed the kubernetes-continue-processing-on-illegal-auth-realm-value branch from 533e684 to b028836 Compare October 12, 2017 13:34
@traefiker traefiker merged commit 8d15840 into traefik:v1.4 Oct 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants