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

UpdateProfileMappingAsync doesn't update properties #618

Closed
gao-artur opened this issue Feb 8, 2023 · 2 comments · Fixed by #623
Closed

UpdateProfileMappingAsync doesn't update properties #618

gao-artur opened this issue Feb 8, 2023 · 2 comments · Fixed by #623
Labels

Comments

@gao-artur
Copy link

Describe the bug?

Hi. I'm trying to add a property mapping to the user profile mapping, but my changes are ignored.

What is expected to happen?

When calling UpdateProfileMappingAsync the profile mapping should be updated in okta, and the response object should contain the updates

What is the actual behavior?

Nothing changes

Reproduction Steps?

Create a SAML Identity Provider and find its profile mapping id (I believe it will happen with any profile mapping for any application or Identity Provider but I didn't test it).

var profileMappingApi = new ProfileMappingApi();
var mapping = await profileMappingApi.GetProfileMappingAsync("your-mapping-id");

mapping.Properties.Add("userType", new ProfileMappingProperty
{
    PushStatus = ProfileMappingPropertyPushStatus.PUSH,
    Expression = "appuser.firstName" // just for example
});

var updatedMapping = await profileMappingApi.UpdateProfileMappingAsync("your-mapping-id", fullMapping);

Additional Information?

The same scenario works in Postman.

.NET Version

7.0.101

SDK Version

6.0.2

OS version

19045 Microsoft Windows 10 Pro 64-bit 10.0.19045

@gao-artur gao-artur added the bug label Feb 8, 2023
@laura-rodriguez
Copy link
Collaborator

Hi @gao-artur,

Thanks for reporting this issue. Our team will review it and prioritize it accordingly.

Internal Ref: OKTA-576010

@laura-rodriguez
Copy link
Collaborator

Fixed in 6.0.7

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 a pull request may close this issue.

2 participants