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

Oauth multi resource support #8642

Closed
wants to merge 1 commit into from

Conversation

MarcialRosales
Copy link
Contributor

@MarcialRosales MarcialRosales commented Jun 21, 2023

Proposed Changes

see #7746

Types of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Bug fix (non-breaking change which fixes issue #NNNN)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause an observable behavior change in existing systems)
  • Documentation improvements (corrections, new content, etc)
  • Cosmetic change (whitespace, formatting, etc)
  • Build system and/or CI

Tasks

  • Use keycloak for selenium tests because it is easier to configure TLS for jwks
  • Clear oidc client library's state either just before redirecting the user to the authorization server or when an error occurs
  • Take into account initiated_login_type . Atm it assumes it is sp_initiated from the selection box
  • Include basic authentication to the initial page if enabled
  • Rename all selenium's .env files to env to facilitate merging in some editing tools
  • Refactored all components' start_ and init_ functions out from suite_template
  • Write documentation : Allow Oauth2 and basic auth for management UI and support multiple oauth2 resources rabbitmq-website#1710
  • Replace rabbitmqctl command add_uaa_key with add_signing_key. Keep àdd_uaa_key` around for at least a release or two. Announce in release notes that it will be soon deprecated. The command's description also says that it is deprecated although it is fully functional.

@mergify mergify bot added the bazel label Jun 21, 2023
@MarcialRosales MarcialRosales self-assigned this Jun 21, 2023
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch from af7b4e9 to ee8120e Compare June 21, 2023 15:09
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch 4 times, most recently from b74a5b3 to 53caf21 Compare July 25, 2023 18:09
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch 2 times, most recently from 1028c24 to 489e457 Compare July 27, 2023 16:22
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch 6 times, most recently from ac4b66d to a2fb08a Compare August 10, 2023 08:53
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch 2 times, most recently from 8703dcf to 71c18e9 Compare August 16, 2023 10:20
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch 3 times, most recently from fbeb810 to 8ad8798 Compare August 18, 2023 11:37
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch 2 times, most recently from 96a2705 to 1b9b475 Compare November 29, 2023 08:40
@MarcialRosales MarcialRosales marked this pull request as ready for review November 29, 2023 09:10
@MarcialRosales MarcialRosales force-pushed the oauth-multi-resource-support branch from 1b9b475 to 2bb1a4c Compare November 29, 2023 13:16
@MarcialRosales
Copy link
Contributor Author

Deprecated by this other PR which fully implemented the multi-oauth feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant