fix(gotrue): Fix the issue where verfiyOTP emits signIn instead of passwordRecovery auth event. (supabase#774)

point-source · web-flow · commit fc426134fff8 · 2023-12-30T23:31:12.000+09:00
Fix supabase#773: verfiyOTP fails to emit AuthChangeEvent.passwordRecovery This PR fixes issue supabase#773: verifyOTP should emit AuthChangeEvent.passwordRecovery instead of AuthChangeEvent.signedIn when OtpType.recovery is passed
diff --git a/packages/gotrue/lib/src/gotrue_client.dart b/packages/gotrue/lib/src/gotrue_client.dart
@@ -509,7 +509,9 @@ class GoTrueClient {
     }
 
     _saveSession(authResponse.session!);
-    notifyAllSubscribers(AuthChangeEvent.signedIn);
+    notifyAllSubscribers(type == OtpType.recovery
+        ? AuthChangeEvent.passwordRecovery
+        : AuthChangeEvent.signedIn);
 
     return authResponse;
   }

Original file line number	Diff line number	Diff line change
`@@ -509,7 +509,9 @@ class GoTrueClient {`
`509`	`509`	`}`
`510`	`510`
`511`	`511`	`_saveSession(authResponse.session!);`
`512`		`- notifyAllSubscribers(AuthChangeEvent.signedIn);`
	`512`	`+ notifyAllSubscribers(type == OtpType.recovery`
	`513`	`+ ? AuthChangeEvent.passwordRecovery`
	`514`	`+ : AuthChangeEvent.signedIn);`
`513`	`515`
`514`	`516`	`return authResponse;`
`515`	`517`	`}`