feat: Authentication Listener knows the difference between a Connect and Auth login

Author: WispyMouse

Assets/Scripts/StandardSamples/Services/AchievementsService.cs

@@ -107,7 +107,7 @@ private static string ProductUserIdToString(ProductUserId productUserId)
             return buffer;
         }
 
-        protected async override void OnLoggedIn()
+        protected async override void OnLoggedIn(AuthenticationListener.AuthenticationLevelChangeType changeType)
         {
             await RefreshAsync();
         }

Assets/Scripts/StandardSamples/Services/EOSService.cs

@@ -159,30 +159,32 @@ protected bool TryGetProductUserId(out ProductUserId productUserId)
 
         }
 
-        private void OnAuthenticationChanged(bool authenticated)
+        private void OnAuthenticationChanged(bool authenticated, AuthenticationListener.AuthenticationLevelChangeType changeType)
         {
             if (authenticated)
             {
-                OnLoggedIn();
+                OnLoggedIn(changeType);
             }
             else
             {
-                OnLoggedOut();
+                OnLoggedOut(changeType);
             }
         }
 
         /// <summary>
         /// Implement this method to perform tasks when a user authenticates.
         /// By default, there is no action taken.
         /// </summary>
-        protected virtual void OnLoggedIn() { }
+        /// <param name="changeType">The type of authentication change.</param>
+        protected virtual void OnLoggedIn(AuthenticationListener.AuthenticationLevelChangeType changeType) { }
 
         /// <summary>
         /// If there are tasks that need to be done when logged out, consider
         /// overriding the Reset() function as that is where such things should
         /// be done.
         /// </summary>
-        protected void OnLoggedOut()
+        /// <param name="changeType">The type of authentication change.</param>
+        protected void OnLoggedOut(AuthenticationListener.AuthenticationLevelChangeType changeType)
         {
             Reset();
         }

Assets/Scripts/StandardSamples/Services/PlayerDataStorageService.cs

@@ -372,9 +372,12 @@ private void FinishFileUpload(string fileName, Action fileUploadCallback = null)
         /// <list type="bullet">
         ///     <item><description><c>QueryFileList()</c></description></item>
         /// </list>
-        protected override void OnLoggedIn()
+        protected override void OnLoggedIn(AuthenticationListener.AuthenticationLevelChangeType changeType)
         {
-            QueryFileList();
+            if (changeType == AuthenticationListener.AuthenticationLevelChangeType.Connect)
+            {
+                QueryFileList();
+            }
         }
 
         protected override Task InternalRefreshAsync()

Assets/Scripts/StandardSamples/Services/StatsService.cs

@@ -94,7 +94,7 @@ private static void Log(string toPrint)
             UnityEngine.Debug.Log(toPrint);
         }
 
-        protected async override void OnLoggedIn()
+        protected async override void OnLoggedIn(AuthenticationListener.AuthenticationLevelChangeType changeType)
         {
             if (TryGetProductUserId(out ProductUserId userId))
             {

Assets/Scripts/StandardSamples/UI/Common/SampleMenu.cs

@@ -162,7 +162,9 @@ protected SampleMenu(bool startsHidden = true, bool requiresAuthentication = tru
         /// <param name="authenticated">
         /// True if the state has changed to authenticated, false otherwise.
         /// </param>
-        private void OnAuthenticationChanged(bool authenticated)
+        /// <param name="authenticationChangeType">
+        /// What kind of authentication change this is.</param>
+        private void OnAuthenticationChanged(bool authenticated, AuthenticationListener.AuthenticationLevelChangeType authenticationChangeType)
         {
             if (authenticated || !RequiresAuthentication)
             {

com.playeveryware.eos/Runtime/Core/AuthenticationListener.cs

@@ -34,6 +34,15 @@ namespace PlayEveryWare.EpicOnlineServices
     /// </summary>
     public class AuthenticationListener: IAuthInterfaceEventListener, IConnectInterfaceEventListener, IDisposable
     {
+        /// <summary>
+        /// Identifies the kind of authentication change.
+        /// </summary>
+        public enum AuthenticationLevelChangeType
+        {
+            Auth,
+            Connect
+        }
+
         /// <summary>
         /// Used to describe functions that handle change in authentication
         /// state.
@@ -42,7 +51,7 @@ public class AuthenticationListener: IAuthInterfaceEventListener, IConnectInterf
         /// True if the authentication state has changed to authenticated, False
         /// otherwise.
         /// </param>
-        public delegate void AuthenticationChangedEventHandler(bool authenticated);
+        public delegate void AuthenticationChangedEventHandler(bool authenticated, AuthenticationLevelChangeType changeType);
 
         /// <summary>
         /// Event that triggers when the state of authentication has changed.
@@ -104,7 +113,8 @@ public bool IsAuthenticated
         /// </summary>
         /// <param name="attemptedState"></param>
         /// <param name="attemptResult"></param>
-        private void TriggerAuthenticationChangedEvent(bool attemptedState, Result attemptResult)
+        /// <param name="changeType">The type of authentication change.</param>
+        private void TriggerAuthenticationChangedEvent(bool attemptedState, Result attemptResult, AuthenticationLevelChangeType changeType)
         {
             // If the attempt to change the state of authentication did not 
             // succeed, then log a warning and stop.
@@ -119,7 +129,7 @@ private void TriggerAuthenticationChangedEvent(bool attemptedState, Result attem
 
             // Trigger the event indicating that the state of authentication for 
             // the user has changed.
-            AuthenticationChanged?.Invoke(attemptedState);
+            AuthenticationChanged?.Invoke(attemptedState, changeType);
         }
 
         /// <summary>
@@ -130,7 +140,7 @@ private void TriggerAuthenticationChangedEvent(bool attemptedState, Result attem
         /// </param>
         public void OnAuthLogin(LoginCallbackInfo loginCallbackInfo)
         {
-            TriggerAuthenticationChangedEvent(true, loginCallbackInfo.ResultCode);
+            TriggerAuthenticationChangedEvent(true, loginCallbackInfo.ResultCode, AuthenticationLevelChangeType.Auth);
         }
 
         /// <summary>
@@ -141,7 +151,7 @@ public void OnAuthLogin(LoginCallbackInfo loginCallbackInfo)
         /// </param>
         public void OnAuthLogout(LogoutCallbackInfo logoutCallbackInfo)
         {
-            TriggerAuthenticationChangedEvent(false, logoutCallbackInfo.ResultCode);
+            TriggerAuthenticationChangedEvent(false, logoutCallbackInfo.ResultCode, AuthenticationLevelChangeType.Auth);
         }
 
         /// <summary>
@@ -152,7 +162,7 @@ public void OnAuthLogout(LogoutCallbackInfo logoutCallbackInfo)
         /// </param>
         public void OnConnectLogin(Epic.OnlineServices.Connect.LoginCallbackInfo loginCallbackInfo)
         {
-            TriggerAuthenticationChangedEvent(true, loginCallbackInfo.ResultCode);
+            TriggerAuthenticationChangedEvent(true, loginCallbackInfo.ResultCode, AuthenticationLevelChangeType.Connect);
         }
 
         /// <summary>