fix(rtn-web-browser): signInWithRedirect needs to be called twice on …

…Android

* fix(rtn-web-browser): signInWithRedirect needs to be called twice on Android

* chore: add code owner

* address feedback

* update codeowners

* update codeowners

* Update .github/CODEOWNERS

---------

Co-authored-by: Ashwin Kumar <ashwsrir@amazon.com>

.github/CODEOWNERS

@@ -27,6 +27,7 @@
 /packages/core/src/clients/internal @ukhan-amazon @haverchuck @cshfang @jimblanc @HuiSF
 /packages/core/src/Hub @ukhan-amazon @haverchuck @cshfang @jimblanc @HuiSF
 /packages/adapter-nextjs @ukhan-amazon @haverchuck @cshfang @jimblanc @HuiSF
+/packages/rtn-web-browser @ukhan-amazon @haverchuck @cshfang @jimblanc @HuiSF
 /packages/storage/src/providers/s3/apis/internal @ukhan-amazon @haverchuck @cshfang @jimblanc @HuiSF
 /packages/storage/src/providers/s3/apis/server @ukhan-amazon @haverchuck @cshfang @jimblanc @HuiSF
 /packages/api-rest/src/apis/server.ts @ukhan-amazon @haverchuck @cshfang @jimblanc @HuiSF

packages/rtn-web-browser/src/apis/openAuthSessionAsync.ts

@@ -61,34 +61,44 @@ const openAuthSessionAndroid = async (url: string, redirectUrls: string[]) => {
 
 		return redirectUrl;
 	} finally {
-		appStateListener?.remove();
-		redirectListener?.remove();
-		appStateListener = undefined;
-		redirectListener = undefined;
+		removeAppStateListener();
+		removeRedirectListener();
 	}
 };
 
 const getAppStatePromise = (): Promise<null> =>
 	new Promise(resolve => {
-		appStateListener = AppState.addEventListener('change', nextAppState => {
-			// if current state is null, the change is from initialization
-			if (AppState.currentState === null) {
-				return;
-			}
+		// remove any stray listeners before creating new ones
+		removeAppStateListener();
 
-			if (nextAppState === 'active') {
-				appStateListener?.remove();
-				appStateListener = undefined;
+		let previousState = AppState.currentState;
+		appStateListener = AppState.addEventListener('change', nextAppState => {
+			if (previousState !== 'active' && nextAppState === 'active') {
+				removeAppStateListener();
 				resolve(null);
 			}
+			previousState = nextAppState;
 		});
 	});
 
 const getRedirectPromise = (redirectUrls: string[]): Promise<string> =>
 	new Promise(resolve => {
+		// remove any stray listeners before creating new ones
+		removeRedirectListener();
+
 		redirectListener = Linking.addEventListener('url', event => {
 			if (redirectUrls.some(url => event.url.startsWith(url))) {
 				resolve(event.url);
 			}
 		});
 	});
+
+const removeAppStateListener = () => {
+	appStateListener?.remove();
+	appStateListener = undefined;
+};
+
+const removeRedirectListener = () => {
+	redirectListener?.remove();
+	redirectListener = undefined;
+};