-
Notifications
You must be signed in to change notification settings - Fork 43
[feature] oidc authentication for NPMplus web admin interface #1664
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
Comments
And forget: I have 7 instances of NPM... each time to create manually user is annoying, controls access to instances impossible. But with SSO will bet easy to do that. |
It is completely untested by me, but if you want you can try this tag of NPMplus: ghcr.io/zoeyvid/npmplus:pr-1668 |
sure) will test it. Thank you so much |
Got this error, nginx doesn't work, can't open any site. |
you can repull the image and retry in around 5 minutes |
ok, please try again with the newest build which should be ready in a few minutes |
Error gone, looks good. Give me couple of days to test it with different SSO. Now looks really good! THANK YOU! |
Is something ligged in the docker logs? |
If you tell me how to turn on the debug logs, will send debug info to you This is how it should be:
|
can you try if it maybe works now? |
Got this:
|
if you want you can test again |
@Zoey2936 I want) I would like to help. You built the wonderful product) Let me check |
Nothing changed, same image:
|
Ohh, the image was not build because of a merge conföict, dhoild be built now |
Should be fixed, I think |
|
Did you removed the domain or is it missing? |
I removed it |
|
Ok, I will check this later |
do you use openid or oauth? |
OAuth2/OpenID Provider - shows in Authentik But looks like OAuth 2.0 provider |
ok since I in the upgrade to v6 of the client I only mad eit work with openid, not oauth, will try to make it alsow ork with oauth |
NPM with this PR works without any issue with the same settings, even same application in Authentik. |
Yes I know, I will adjust it to work with oauth too |
Any luck?) |
sorry, I had no time yeet |
No worries) I will wait. Take your time. |
ok, you can now test again |
I can't login ether with email or with SSO.
|
ok you can try again please |
There is no new image... |
The last build should be 14 hours old: #1668 (comment) |
Able to login with email/password, but OIDC: [OIDC ] › ℹ info Initializing OAuth flow |
ok you can try again |
Same issue, but now nothing on the page, only in logs:
|
Sorry for bothering you, but do you have any updates? |
I think, I will revert back to v5 and will later try to migrate to v6 to get it faster merged, will work on it as soon as possible |
This PR works for me. Could you tell me v5 and v6, what do you mean? |
With v5/v6 I mean the version of the nodejs openid-client package |
will be included in the next release |
v6 shows me the same error as before. v5 broken. Add comments to PRs |
As we discuss could you implement NginxProxyManager#4010 this in to NPMplus.
This helps a lot in env and a lot of users waiting on this feature.
Please)
The text was updated successfully, but these errors were encountered: