-
Notifications
You must be signed in to change notification settings - Fork 152
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
feat: make OAuth2 configuration optional #1057
base: master
Are you sure you want to change the base?
Conversation
@amvanbaren Here is my response to the comment #1056 (comment)
The configuration for open-vsx.org and other deployments includes OAuth2. The UI will remain unchanged since But for the new instances only if they will be configured without OAuth2 configuration, Publish and Login buttons would not be displayed in the UI by default. And it makes it possible to run OpenVSX instance without dummy GH OAuth2 configuration. About a slight glitch, I tested current changes on gitpod and this is what I see: no_oauth.mp4When OAuth2 is NOT configured in application.yaml and application-ovsx.properties no_oauth.mp4How can I switch to the dark theme? |
By selecting dark mode in your browser. |
Signed-off-by: Valeriy Svydenko <vsvydenk@redhat.com>
821acc9
to
08da93d
Compare
I can see the glitch after refreshing now: screencast-nimbusweb_me-2024_12_10-15_15_52.mp4I'll try to find a way how to fix that. @amvanbaren do you happen to have any suggestions or ideas? |
@amvanbaren I ran the OpenVSX registry without my changes from the master branch on gitpod and I see the same behavior. It seems the issue is not caused by my changes: screencast-nimbusweb_me-2024_12_10-16_33_57.mp4 |
@svor I've narrowed the scope of this PR by checking if a user can login instead of whether oauth is enabled. |
@amvanbaren Thank you, changes look good! With OAuth: screencast-nimbusweb_me-2024_12_11-13_59_47.mp4 |
Hell @amvanbaren, |
This pull request introduces several changes to enable and check OAuth2 support in the application. The key changes include adding methods to check OAuth2 status, modifying the security configuration, and updating the web UI to reflect OAuth2 status.
In case when OAuth2 is not configured Publish and Log In buttons are hidden and we have like readonly mode:
Related issue: #1041
Web version:
Mobile version:
What I did to test how it works