fix: Move on_session_creation hook after session is created

frappe/auth.py

@@ -125,9 +125,6 @@ class LoginManager:
 			if self.login() is False:
 				return
 			self.resume = False
-
-			# run login triggers
-			self.run_trigger("on_session_creation")
 		else:
 			try:
 				self.resume = True
@@ -233,6 +230,8 @@ class LoginManager:
 		self.user = frappe.local.session_obj.user
 		frappe.local.session = frappe.local.session_obj.data
 		self.clear_active_sessions()
+		if not resume:
+			self.run_trigger("on_session_creation")
 
 	def clear_active_sessions(self):
 		"""Clear other sessions of the current user if `deny_multiple_sessions` is not set"""

frappe/utils/oauth.py

@@ -205,7 +205,6 @@ def login_oauth_user(
 		)
 
 	frappe.local.login_manager.login_as(user)
-	frappe.local.login_manager.run_trigger("on_session_creation")
 
 	# because of a GET request!
 	frappe.db.commit()