Merge pull request #31410 from gavindsouza/oauth-on_session_creation

fix: Run on_session_creation on OAuth logins

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

@@ -204,8 +204,7 @@ def login_oauth_user(
 			http_status_code=403,
 		)
 
-	frappe.local.login_manager.user = user
-	frappe.local.login_manager.post_login()
+	frappe.local.login_manager.login_as(user)
 
 	# because of a GET request!
 	frappe.db.commit()