fix: move server_port assignment inside try block

saikumarvasa100-hash · web-flow · commit baacc4a3cf4b · 2025-11-14T09:56:40.000+05:30
Fixes AttributeError by ensuring server_port is accessed only after successful HTTPServer creation
diff --git a/ggshield/verticals/auth/oauth.py b/ggshield/verticals/auth/oauth.py
@@ -164,6 +164,7 @@ def _prepare_server(self) -> None:
                 functools.partial(RequestHandler, self),
             )
             self._port = self.server.server_port  # Get the assigned port
+            
         except OSError as e:
             raise UnexpectedError(f"Could not create local server: {e}")
 

Original file line number	Diff line number	Diff line change
`@@ -164,6 +164,7 @@ def _prepare_server(self) -> None:`
`164`	`164`	`functools.partial(RequestHandler, self),`
`165`	`165`	`)`
`166`	`166`	`self._port = self.server.server_port # Get the assigned port`
	`167`	`+`
`167`	`168`	`except OSError as e:`
`168`	`169`	`raise UnexpectedError(f"Could not create local server: {e}")`
`169`	`170`