-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
add header authentication support for the web-ui #16558
base: master
Are you sure you want to change the base?
Conversation
hey @lukasz-walkiewicz could you ptal when you get the chance. wrote tests as you asked in #14450 |
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
👋 @kenandominic - this PR has become inactive. We hope you are still interested in working on it. Please let us know, and we can try to get reviewers to help with that. We're working on closing out old and inactive PRs, so if you're too busy or this has too many merge conflicts to be worth picking back up, we'll be making another pass to close it out in a few weeks. |
I'm also interested in this feature. @hashhar @lukasz-walkiewicz what is outstanding to merge this? |
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
hey @mosabua, I'll spend some time again, hopefully later today, to resolve the merge conflicts if you can get someone to review it. |
hey @huw0, you can apply the changes in this PR if you have a fork of trino. we've (salesforce) been using header authn to guard our UI for nearly a year now |
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
2 similar comments
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
hey @mosabua I have resolved all the conflicts |
@cla-bot check |
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
The cla-bot has been summoned, and re-checked this pull request! |
Can you submit a signed CLA and check the build failures out @kenandominic ? |
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
1 similar comment
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Kenan Dominic.
|
hey @mosabua I was able to fix the issue with the cla check not passing. all tests pass except |
Thanks for the CLA and such @kenandominic . The test results coming from the Delta Lake test are probably just a CI failure. Every new push will trigger the whole build again but I would wait until we get further review comments. |
Can @lukasz-walkiewicz @hashhar or maybe @dain help here? |
Great to hear that this has been battle tested! Is there anything that can be done to progress this PR? |
This pull request has gone a while without any activity. Tagging the Trino developer relations team: @bitsondatadev @colebow @mosabua |
Closing this pull request, as it has been stale for six weeks. Feel free to re-open at any time. |
Reopening .. we still want this feature to get in. Adding stale-ignore label. |
Hi @mosabua, @lukasz-walkiewicz - would be great to get this in soon if possible? |
@wendigo @koszti .. you are working on this a bit recently. Any input? Also @kenandominic can you change commit message to "Add header authentication support to the Web UI" |
@@ -46,6 +46,7 @@ The following Web UI authentication types are also supported: | |||
- `KERBEROS`, see details in {doc}`/security/kerberos` | |||
- `JWT`, see details in {doc}`/security/jwt` | |||
- `OAUTH2`, see details in {doc}`/security/oauth2` | |||
- `HEADER`, see details in {doc}`/develop/header-authenticator` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to get user facing docs for header authentication going as well at some stage.
Description
Installs the header authenticator and header authenticator manager modules in the web UI authentication module
relevant configuration
web-ui.authentication.type=header
in place of #14450
Additional context and related issues
Release notes
( ) This is not user-visible or docs only and no release notes are required.
(x) Release notes are required, please propose a release note for me.
( ) Release notes are required, with the following suggested text: