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

No error when importing authorization data of duplicated ID #15051

Open
hokuda opened this issue Oct 20, 2022 · 2 comments
Open

No error when importing authorization data of duplicated ID #15051

hokuda opened this issue Oct 20, 2022 · 2 comments
Labels
area/authorization-services Indicates an issue on Authorization area kind/enhancement Categorizes a PR related to an enhancement

Comments

@hokuda
Copy link
Contributor

hokuda commented Oct 20, 2022

Describe the bug

When importing authorization data into a client, even if same id's of resources/scopes/policies/permissions exists in the other client, Keycloak shows no error.
But resources/scopes/policies/permissions are not imported actually.

Version

19.0.3

Expected behavior

Shows an error.

Actual behavior

Shows no error.

How to Reproduce?

  1. Import test-realm-1.json
  2. Import test-client-1.json
  3. Import test-client-2.json
  4. Import test-authz-config.json into test-client-2

Anything else?

No response

@hokuda hokuda added kind/bug Categorizes a PR related to a bug status/triage labels Oct 20, 2022
@hokuda
Copy link
Contributor Author

hokuda commented Oct 20, 2022

jsons.tar.gz

@stianst stianst added the area/authorization-services Indicates an issue on Authorization area label Oct 24, 2022
@pedroigor pedroigor added kind/enhancement Categorizes a PR related to an enhancement and removed kind/bug Categorizes a PR related to a bug status/triage labels Jan 9, 2023
@pedroigor pedroigor removed their assignment Jan 9, 2023
@pedroigor
Copy link
Contributor

We need to make sure the returned scope/policy/resource is actually associated with the client we are importing. The findById methods should take this into account.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/authorization-services Indicates an issue on Authorization area kind/enhancement Categorizes a PR related to an enhancement
Projects
None yet
Development

No branches or pull requests

3 participants