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

Changing the PRIMARY domain causes SCIM failures #13325

Closed
mifrazmurthaja opened this issue Mar 24, 2022 · 1 comment
Closed

Changing the PRIMARY domain causes SCIM failures #13325

mifrazmurthaja opened this issue Mar 24, 2022 · 1 comment
Assignees
Milestone

Comments

@mifrazmurthaja
Copy link

mifrazmurthaja commented Mar 24, 2022

Describe the issue:
The PRIMARY domain name can be changed, by adding the configuration below in deployment.toml file.

[user_store.properties]
DomainName = "TEST.COM"

However, regardless of the domain configured, the domain name will be taken as PRIMARY at getFilteredDomainName () when enable_filtering_enhancements or filter_users_and_groups_from_primary_domain is enabled.

[scim2]
enable_filtering_enhancements = true

[scim2]
filter_users_and_groups_from_primary_domain = true

This worked fine for SCIM requests regardless of the invalid domain. However, with carbon-kernel/pull/2860, this has broken and the SCIM requests are failing with filtering parameters.

How to reproduce:

  1. Change the PRIMARY domain and add any of the above configurations.
  2. Invoke the SCIM request below in the latest update level of IS 5.10 (tested: 5.10.0.127).
    https://localhost:9443/scim2/Users?filter=username+eq+admin
    
  3. It'll return with the status code: 500 with the response below.
    {
    "schemas": [
        "urn:ietf:params:scim:api:messages:2.0:Error"
    ],
    "detail": "Error while filtering the users for filter with attribute name: urn:ietf:params:scim:schemas:core:2.0:User:userName , filter operation: eq and attribute value: PRIMARY/admin. ",
    "status": "500"
    }
    

Environment information

  • Product Version: IS 5.10.0
@ShanChathusanda93
Copy link
Contributor

Closing the issue since the public fix is done.

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

No branches or pull requests

3 participants