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

SCIM user creation works when providing invalid origin #2576

Closed
robl-sap opened this issue Oct 26, 2023 · 4 comments · Fixed by #2688
Closed

SCIM user creation works when providing invalid origin #2576

robl-sap opened this issue Oct 26, 2023 · 4 comments · Fixed by #2688
Assignees

Comments

@robl-sap
Copy link
Contributor

robl-sap commented Oct 26, 2023

What version of UAA are you running?

76.21.0 , issue is also present in older versions

How are you deploying the UAA?

Via bosh release. However, issue is independent of deployment

What did you do?

Create or update a shadow user with invalid origin.

curl -X POST 'https://<uaaurl>/Users' -H "Authorization: bearer <admintoken>" -H "Content-type: application/json" -d '{"userName": "user-does-not-exist",  "origin": "does-not-exist", "emails": [{"value": [[email protected]](mailto:[email protected]), "primary": false}]}'

What did you expect to see? What goal are you trying to achieve with the UAA?

Creation or update is rejected because the origin does not map to a known identity provider.

What did you see instead?

User is created or updated.

@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/186341652

The labels on this github issue will be updated when the story is started.

@klaus-sap
Copy link
Contributor

Also mentioned here: #2505 -> Proposal 6.

@klaus-sap
Copy link
Contributor

To avoid incompatible changes, we may add a switch to the IdZ configuration if the origin shall be checked when creating or updating users

@klaus-sap
Copy link
Contributor

PR: #2688

@strehle strehle linked a pull request Jan 30, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
4 participants