FunctionMonkey icon indicating copy to clipboard operation
FunctionMonkey copied to clipboard

SignalR Negotiate: access token lifetime

Open alexeymarkov opened this issue 5 years ago • 2 comments

Hi James,

I found in the MS implementation that the default access token lifetime is 1 hour: ServiceOptions.cs Constants.cs

// Default access token lifetime public static readonly TimeSpan DefaultAccessTokenLifetime = TimeSpan.FromHours(1);

Could you please change it also from 30 minutes to 1 hour in your implementation?

Or better make it configurable or utilize SignalR ServiceOptions. Sure, if MS opened their AzureSignalRClient it would be perfect (maybe it is worth contacting the Azure team...).

Regards, Alexey

alexeymarkov avatar Feb 03 '20 10:02 alexeymarkov

I just found out this is already possible to generate an instance of SignalRConnectionInfo using existing SignalR extension implementation. Have a look at the sample.

alexeymarkov avatar Feb 03 '20 10:02 alexeymarkov

Cool, I’ll get that included! Thanks!

Get Outlook for iOShttps://aka.ms/o0ukef


From: alexeymarkov [email protected] Sent: Monday, February 3, 2020 10:29:05 AM To: JamesRandall/FunctionMonkey [email protected] Cc: Subscribed [email protected] Subject: Re: [JamesRandall/FunctionMonkey] SignalR Negotiate: access token lifetime (#123)

I just found out this is already possible to generate an instance of SignalRConnectionInfo using existing SignalR extension implementation. Have a look at the samplehttps://github.com/Azure/azure-functions-signalrservice-extension/blob/dev/samples/simple-chat/csharp/FunctionApp/Functions.cs#L33l.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/JamesRandall/FunctionMonkey/issues/123?email_source=notifications&email_token=AABZVV72WHHYRJQ75BEJ6DTRA7WXDA5CNFSM4KPCSQZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKTJSAY#issuecomment-581343491, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABZVV4EQDSRZMTJUZPJEHLRA7WXDANCNFSM4KPCSQZQ.

JamesRandall avatar Feb 03 '20 11:02 JamesRandall