@alexanderpilhar @ismcagdas
It seems to me that the onclose() only fires once and not again when connection.start() fails. This is using SignalR Core, is it supposed to fire onclose() again in signalr core?
Actually, I believe that the onClose event only fires if the connection actually successfully connected. So if the initial connection was successful but then closed, you will try to open the connection once but then never try again.
See better reconnection logic here from David Fowler: https://github.com/davidfowl/UT3/blob/dac409886c1bb7aec7c150b74d4ce9a3e246f03c/UTT/wwwroot/js/utt.js#L141-L153
@ismcagdas Wouldn't setInterval be needed to check every 5 seconds instead of setTimeout?
@ismcagdas It seems to me the autoconnect tries to connect one time when hitting the onclose event, but if it fails to connect then it doens't try to connect again.
Is there some recommended way to do this?
Also I noticed in the chat-signalr.service init method it calls abp.signalr.connect(); and then abp.signalr.startConnection. Could you explain a little what the need for both of them are, or how they work?
I noticed in the default project that abp.signalr.autoconnect is set to false in ngAfterViewInit() of app.component.ts.
I want to change this for my app to autoconnect, but just wanted to make sure that there isn't an important reason for setting it to false before making that change.
Are there any gotchas to doing this?
Where is the source code for ExternalAuthManager?
Yes, I would like the ability for a tenant to choose whether to use classic authentication or Azure AD (ADAL).
@klir If you create the issue, I'll give it a thumbs up. I also think the implementation should be multi-tenant aware.
We are building an application on ANZ using multitenancy and we need to use Azure AD for authentication. Our project is Core/Angular. Is this possible? If so, is there any guidance available?
Issue created here: <a class="postlink" href="https://github.com/aspnetzero/aspnet-zero-core/issues/1570">https://github.com/aspnetzero/aspnet-ze ... ssues/1570</a>
