Skip to content

user story #3011011: implement federated login (oidc) #66

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
Open
pikachugb wants to merge 6 commits into
base: master
Choose a base branch
from
Open

user story #3011011: implement federated login (oidc) #66

pikachugb wants to merge 6 commits into from

Conversation

@pikachugb
Copy link
Collaborator

@pikachugb pikachugb commented Jan 19, 2026 •
edited
Loading

https://center.almoctane.com/ui/entity-navigation?p=1001/1002&entityType=work_item&id=3011011

  • in UI:

    • implemented a new UI for OIDC
    • if OIDC is enabled, the user has to configure the OIDC discovery url and the OIDC client id and secret found in octane.sso

  • in the server side:

    • Added method to configure OIDC settings on the server side
    • Fetches the OIDC discovery document to get the token endpoint.
    • Calls the token endpoint with client credentials to get an IDP access token.
    • Exchanges the IDP access token at the Octane /oauth/token endpoint to get an Octane access token.
    • Stores the Octane access token as a cookie for future requests.
    • Integrated automatic OIDC token refresh when authentication fails
image

@pikachugb pikachugb self-assigned this Jan 19, 2026
vladbenchea
vladbenchea reviewed Jan 19, 2026
View reviewed changes
@pikachugb pikachugb force-pushed the dmoldoveanu_story_3011011_fed_login branch from 1ea2232 to 580b3ba Compare January 19, 2026 13:50
RalucaOT
RalucaOT reviewed Jan 19, 2026
View reviewed changes
src/main/java/com/microfocus/octane/plugins/rest/RestConnector.java Outdated

private String getTokenEndpointFromDiscovery() throws IOException {
String discoveryJson = httpGetAbsolute(discoveryUrl);
ObjectMapper mapper = new ObjectMapper();
Copy link
Collaborator

@RalucaOT RalucaOT Jan 19, 2026

Choose a reason for hiding this comment

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

W: would create a single private static final ObjectMapper and reuse it, as it is recreated multiple times

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vladbenchea vladbenchea vladbenchea approved these changes

@RalucaOT RalucaOT RalucaOT approved these changes

@SerbanIonescu SerbanIonescu SerbanIonescu approved these changes

Assignees

@pikachugb pikachugb

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@pikachugb @vladbenchea @RalucaOT @SerbanIonescu