Skip to content

Integrate OAuth flow UI and RBAC roles - #439

Merged
drslebedev merged 1 commit into
mainfrom
oauthintegration
Apr 27, 2026
Merged

Integrate OAuth flow UI and RBAC roles#439
drslebedev merged 1 commit into
mainfrom
oauthintegration

Conversation

@magnusg-kdab

@magnusg-kdab magnusg-kdab commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

This is the followup of fair-acc/opencmw-cpp#402 to integrate the OAuth flow and RBAC in the UI.

Once the user has signed in, available roles will be displayed in the UI. The public key is added to the keystore worker and messages can be signed and in turn other clients can cryptographically verify that the user does indeed have access to the role.

The available roles are configured on the keycloak/oauth-provider side and can be set dynamically depending on provided scope.

Note that the OAuth session does not survive application restarts yet, as there is no persistent cache yet and login is only available in editableMode.

Closes #203

@magnusg-kdab magnusg-kdab self-assigned this Apr 21, 2026
@magnusg-kdab
magnusg-kdab force-pushed the oauthintegration branch 2 times, most recently from a86aa1d to 4364093 Compare April 23, 2026 08:40
Signed-off-by: Magnus Groß <magnus.gross+github@kdab.com>
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

PR produced different images:

blockneighbourspreview_0001.png

Got:

blockneighbourspreview_0001.png

Expected:

blockneighbourspreview_0001.png

Diff:

blockneighbourspreview_0001.png

chart_0001.png

Got:

chart_0001.png

Expected:

chart_0001.png

Diff:

chart_0001.png

chart_fg_dipole_0001.png

Got:

chart_fg_dipole_0001.png

Expected:

chart_fg_dipole_0001.png

Diff:

chart_fg_dipole_0001.png

dashboardpage_layout_0001.png

Got:

dashboardpage_layout_0001.png

Expected:

dashboardpage_layout_0001.png

Diff:

dashboardpage_layout_0001.png

dashboardpage_layout_0002.png

Got:

dashboardpage_layout_0002.png

Expected:

dashboardpage_layout_0002.png

Diff:

dashboardpage_layout_0002.png

dashboardpage_layout_0003.png

Got:

dashboardpage_layout_0003.png

Expected:

dashboardpage_layout_0003.png

Diff:

dashboardpage_layout_0003.png

dashboardpage_layout_0004.png

Got:

dashboardpage_layout_0004.png

Expected:

dashboardpage_layout_0004.png

Diff:

dashboardpage_layout_0004.png

filtercomboboxes_0001.png

Got:

filtercomboboxes_0001.png

Expected:

filtercomboboxes_0001.png

Diff:

filtercomboboxes_0001.png

filtercomboboxes_0002.png

Got:

filtercomboboxes_0002.png

Expected:

filtercomboboxes_0002.png

Diff:

filtercomboboxes_0002.png

filtercomboboxes_0003.png

Got:

filtercomboboxes_0003.png

Expected:

filtercomboboxes_0003.png

Diff:

filtercomboboxes_0003.png

flowgraph_0001.png

Got:

flowgraph_0001.png

Expected:

flowgraph_0001.png

Diff:

flowgraph_0001.png

flowgraph_0002.png

Got:

flowgraph_0002.png

Expected:

flowgraph_0002.png

Diff:

flowgraph_0002.png

keypad_0001.png

Got:

keypad_0001.png

Expected:

keypad_0001.png

Diff:

keypad_0001.png

@magnusg-kdab

Copy link
Copy Markdown
Contributor Author

Note the codacy "warnings" are all false positives: Those class members are all being used.

@drslebedev drslebedev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR!

The changes look good to me.

@drslebedev
drslebedev merged commit aeb4d6d into main Apr 27, 2026
8 of 9 checks passed
@drslebedev
drslebedev deleted the oauthintegration branch April 27, 2026 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[5SP;7SP] UI: Integrate OAuth triggering into the UI

2 participants