Skip to content

Implement the callback for the SignalR connection state. #129

Merged
bnicholesdell merged 1 commit intoOneIdentity:masterfrom
bnicholesdell:addeventlistenercallback
Jul 20, 2023
Merged

Implement the callback for the SignalR connection state. #129
bnicholesdell merged 1 commit intoOneIdentity:masterfrom
bnicholesdell:addeventlistenercallback

Conversation

@bnicholesdell
Copy link
Contributor

Implement the callback for the SignalR connection state. Fix some bugs around connecting to the A2A signalr service and mark the A2A Persist event listener APIs that take a thumbprint as not supported. Connecting and authenticating to the A2A signalr service is handled differently than signalr on the core service. The core service authenticates using an access token via restsharp which is able to use the windows certificate store. It then uses Okhttp with the access token to connect to the signalr service. Connecting to the A2A signalr service is done entirely using okhttp and passing a certificate. Okhttp can't pull a certificate out of the windows certificate store by thumbprint. The problem is difference in the authentication methods when using Okhttp.

…s around connecting to the A2A signalr service and mark the A2A Persist event listener APIs that take a thumbprint as not supported. Connecting and authenticating to the A2A signalr service is handled differently than signalr on the core service. The core service authenticates using an access token via restsharp which is able to use the windows certificate store. It then uses Okhttp with the access token to connect to the signalr service. Connecting to the A2A signalr service is done entirely using okhttp and passing a certificate. Okhttp can't pull a certificate out of the windows certificate store by thumbprint. The problem is difference in the authentication methods when using Okhttp.
@bnicholesdell bnicholesdell merged commit b16db23 into OneIdentity:master Jul 20, 2023
@bnicholesdell bnicholesdell deleted the addeventlistenercallback branch July 20, 2023 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant