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

HTTP Authorization #491

Closed
senthilnathan opened this issue May 11, 2021 · 5 comments · Fixed by #627
Closed

HTTP Authorization #491

senthilnathan opened this issue May 11, 2021 · 5 comments · Fixed by #627

Comments

@senthilnathan
Copy link
Contributor

We are using Knative with Contour and would like to add authorization to the http requests to the knative services. Contour already has some support for this: https://projectcontour.io/docs/main/config/client-authorization/. This needs to be supported and exposed through net-contour.
@senthilnathan senthilnathan mentioned this issue May 11, 2021
Closed
@dprotaso
Copy link
Contributor

This may be related to #347 ?

@senthilnathan
Copy link
Contributor Author

I do see some overlap with that issue. Here is a relevant slack discussion for this requirement: https://knative.slack.com/archives/CBYV1E0TG/p1620686386019800?thread_ts=1620678576.019500&cid=CBYV1E0TG

@dprotaso dprotaso mentioned this issue May 19, 2021
Closed
@github-actions
Copy link

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

@github-actions github-actions bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 10, 2021
@dprotaso dprotaso reopened this Sep 10, 2021
@dprotaso
Copy link
Contributor

/remove-lifecycle stale

@knative-prow-robot knative-prow-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Sep 10, 2021
@senthilnathan
Copy link
Contributor Author

@dprotaso, I would like to help towards implementing this. Any pointers to get started will help.
Do we need to complete knative/pkg#2128 first for this issue?

@senthilnathan senthilnathan mentioned this issue Sep 29, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

HTTP External Auth Support senthilnathan/net-contour
3 participants
@dprotaso @senthilnathan @knative-prow-robot