Skip to content

feat(WD-30480): Migrate auth from U1-SSO to C-IDP#317

Open
muhammad-ali-pk wants to merge 3 commits intomainfrom
WD-30480
Open

feat(WD-30480): Migrate auth from U1-SSO to C-IDP#317
muhammad-ali-pk wants to merge 3 commits intomainfrom
WD-30480

Conversation

@muhammad-ali-pk
Copy link
Copy Markdown
Contributor

@muhammad-ali-pk muhammad-ali-pk commented Mar 27, 2026

Done

  • Replaced U1-SSO with C-IDP
  • Drive by: Optimized taskfile and fixed some gunicorn issues when running project with taskfile

QA

  • Check out this pull request
  • Find assets.ubuntu.com c-idp credentials and directory API token on bitwarden, place them in your .env.local
  • Run the site using the command task
  • Make sure your company VPN is on
  • View the site locally in your web browser at: http://localhost:8017/manager
  • Verify the new auth works as expected
  • Verify all actions are successful

Issue / Card

Fixes WD-30480

@petesfrench
Copy link
Copy Markdown
Contributor

Tried both staging and production credentials but get this error:

The request is missing a required parameter, includes an invalid parameter value, includes a parameter more than once, or is otherwise malformed. The 'redirect_uri' parameter does not match any of the OAuth 2.0 Client's pre-registered redirect urls.

@muhammad-ali-pk
Copy link
Copy Markdown
Contributor Author

muhammad-ali-pk commented Mar 27, 2026

Tried both staging and production credentials but get this error:

The request is missing a required parameter, includes an invalid parameter value, includes a parameter more than once, or is otherwise malformed. The 'redirect_uri' parameter does not match any of the OAuth 2.0 Client's pre-registered redirect urls.

@petesfrench Can you please try accessing the application at localhost:8017/manager instead of 0.0.0.0:8017/manager? The 0.0.0.0 url got populated in the PR description from template. The callback uri registered for dev is localhost, not 0.0.0.0, so the application should be accessed at localhost:8017. Updated in description as well.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants