-
-
Notifications
You must be signed in to change notification settings - Fork 94
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] publish to masstransit fails #554
Comments
Hello :) I believe you received this exception because the SCIMRepresentation object is being passed as a parameter to MassTransit, and it is not possible to serialize this object due to the presence of a reference to the Parent in the SCIMRepresentationAttribute attribute. In the SCIM Server implementation, the SCIMRepresentation is not directly passed to MassTransit. Instead, it is serialized into JSON using the following code before being passed to MassTransit:
Could you please check if you can serialize the Representation before passing the result to MassTransit? KR, SID |
Hello I'm sorry for the incomplete information. The exception was of type
Kind regards |
Thank you for raising the exception, but it should not occur in the SCIM Server. This is because the SCIMRepresentation is directly passed to MassTransit, but before sending it to MassTransit, it is first serialized into JSON. Have you added custom code that sends the SCIMRepresentation instead of the JSON object? Please refer to my previous message for more information :) |
No I did not :( I guess that prior to publishing the event, it will be preferable to not add the representation that is attached to the attribute as well (it happens only for reference events). WDYT? |
Also, can you please add my account as a contributor? I'd like to submit a pull request |
Oops, my bad ! The exception is being thrown because the SCIMRepresentationAttribute is passed as a parameter into the events RepresentationRefAttributeAddedEvent, RepresentationRefAttributeRemovedEvent, and RepresentationRefAttributeUpdatedEvent. Consequently, an infinite loop occurs between SCIMRepresentationAttribute and SCIMRepresentation. I will ignore the Representation attached to the SCIMRepresentationAttribute :) I have sent you an invitation to become a collaborator |
The issue should be fixed in the branch Changeset : 1419d76 |
Thank you very much!
…On Tue, Jul 25, 2023, 22:18 SimpleIdServer ***@***.***> wrote:
The issue should be fixed in the branch release/4.0.2
Changeset : 1419d76
<1419d76>
—
Reply to this email directly, view it on GitHub
<#554 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIB5S7MMSAWQGVIG3IMN633XSALYNANCNFSM6AAAAAA2VIFUE4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Hello I pushed an improvement to release/4.0.2. Is that OK? Kind regards |
Hello, Sorry for my late reply, i'm ok with your changes :) KR, SID |
Hello, how are you?
I'm having issues with publishing events related to data with nested groups.
I'm getting the following error when trying to publish the event.
Do you think you know what the root cause of this?
It was introduced when I upgraded from 2.0.25 to 4.0.2.
Kind regards
Dan
The text was updated successfully, but these errors were encountered: