Skip to content
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

kie-issues#1827: Refactor Management Console to support multiple runtimes and OpenID Connect #2758

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

thiagoelg
Copy link
Member

@thiagoelg thiagoelg commented Nov 21, 2024

Closes apache/incubator-kie-issues#1827

  • Adds support to connect to multiple runtimes at the same time;
  • Support for authenticated runtimes (using OpenID Connect);
  • Refactor/rewrite of enveloped components, using and/or fixing React Hooks;
  • Support for bookmarkable-urls.

nonce and prompt are commented out: apache/incubator-kie-issues#1830


Demo

Screen.Recording.2025-02-13.at.16.12.36.mov

How to test:

  • Build everything with pnpm -r -F runtime-tools-management-console-webapp... build:dev;
  • Run the webapp and the services (2 quarkus apps + Keycloak) with pnpm -F runtime-tools-management-console-webapp start:app;
  • Connect the runtimes to the Management Console (see video above);
  • Create processes:
    • Secured runtime:
      • export SECURED_RUNTIME_IDP_TOKEN=$(\
            curl -X POST http://localhost:9071/realms/management-console-dev-webapp-realm/protocol/openid-connect/token \
            -H 'content-type: application/x-www-form-urlencoded' \
            -d "client_id=management-console-dev-webapp" \
            -d "grant_type=password" \
            -d "username=jdoe" \
            -d "password=jdoe" \
            -d "scope=openid" | jq --raw-output '.access_token' \
         )
        
      • curl -H "Content-Type: application/json" -H "Accept: application/json" -X POST http://localhost:8080/hiring -H "Authorization: Bearer $SECURED_RUNTIME_IDP_TOKEN" -d '{"candidateData": { "name": "Jon", "lastName": "Snow", "email": "[email protected]", "experience": 5, "skills": ["Java", "Kogito", "Fencing"]}}'
        
    • Unsecured runtime:
      • curl -H "Content-Type: application/json" -H "Accept: application/json" -X POST http://localhost:8081/hiring -d '{"candidateData": { "name": "Jon", "lastName": "Snow", "email": "[email protected]", "experience": 5, "skills": ["Java", "Kogito", "Fencing"]}}'
        

@tiagobento tiagobento added area:bpmn pr: DO NOT MERGE Draft PR, not ready for merging labels Nov 21, 2024
@tiagobento
Copy link
Contributor

@thiagoelg I'm afraid this is blocked by apache/incubator-kie-issues#1575 and by us defining usage of quarkus-oidc-proxy as auth backend for Management Console runtimes...

@thiagoelg thiagoelg changed the title NO-ISSUE: Refactor Management Console to support multiple runtimes and OpenID Connect kie-issues#1827: Refactor Management Console to support multiple runtimes and OpenID Connect Feb 12, 2025
@tiagobento tiagobento requested a review from jomarko February 13, 2025 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:bpmn pr: DO NOT MERGE Draft PR, not ready for merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Management Console: Add support for OIDC authentication and multiple Processes runtimes
2 participants