Differentiate error string

Author: hilarex

trapster/modules/base.py

@@ -1,4 +1,5 @@
 import asyncio
+import errno
 import logging
 from typing import Optional
 
@@ -61,8 +62,16 @@ def __init__(self, config, logger, bindaddr="0.0.0.0", **kwargs):
         self.server = None
         self.task = None
 
-    def _log_bind_error(self):
-        logging.error(f"Service {self.service_name} could not be started on {self.bindaddr}:{self.port}: address already in use")
+    def _log_bind_error(self, exc: Exception = None):
+        if isinstance(exc, OSError) and exc.errno == errno.EACCES:
+            reason = f"permission denied"
+        elif isinstance(exc, OSError) and exc.errno == errno.EADDRINUSE:
+            reason = "address already in use"
+        elif isinstance(exc, OSError):
+            reason = str(exc)
+        else:
+            reason = "address already in use"
+        logging.error(f"Service {self.service_name} could not be started on {self.bindaddr}:{self.port}: {reason}")
 
     async def start(self):
         # Start the server in a separate task
@@ -77,8 +86,8 @@ async def _start_server(self):
             await self.server.serve_forever()
         except asyncio.CancelledError:
             raise
-        except OSError:
-            self._log_bind_error()
+        except OSError as e:
+            self._log_bind_error(e)
             return False
         except Exception as e:
             logging.error(e)

trapster/modules/dns.py

@@ -115,8 +115,8 @@ async def _start_server(self):
             await self.server.serve_forever()
         except asyncio.CancelledError:
             raise
-        except OSError:
-            self._log_bind_error()
+        except OSError as e:
+            self._log_bind_error(e)
             return False
         except Exception as e:
             logging.error(e)

trapster/modules/http.py

@@ -486,8 +486,8 @@ async def _start_server(self):
         self.server = uvicorn.Server(config)
         try:
             await self.server.serve()
-        except (OSError, SystemExit):
-            self._log_bind_error()
+        except (OSError, SystemExit) as e:
+            self._log_bind_error(e)
             return False
         except asyncio.CancelledError:
             self.server.should_exit = True

trapster/modules/https.py

@@ -73,8 +73,8 @@ async def _start_server(self):
         self.server = uvicorn.Server(config)
         try:
             await self.server.serve()
-        except (OSError, SystemExit):
-            self._log_bind_error()
+        except (OSError, SystemExit) as e:
+            self._log_bind_error(e)
             return False
         except asyncio.CancelledError:
             self.server.should_exit = True

trapster/modules/ldaps.py

@@ -74,8 +74,8 @@ async def _start_server(self):
             await self.server.serve_forever()
         except asyncio.CancelledError:
             raise
-        except OSError:
-            self._log_bind_error()
+        except OSError as e:
+            self._log_bind_error(e)
             return False
         except Exception as e:
             logging.error(e)

trapster/modules/snmp.py

@@ -71,8 +71,8 @@ async def _start_server(self):
                                         local_addr=(self.bindaddr, self.port))
         except asyncio.CancelledError:
             raise
-        except OSError:
-            self._log_bind_error()
+        except OSError as e:
+            self._log_bind_error(e)
             return False
         except Exception as e:
             logging.error(e)

trapster/modules/ssh.py

@@ -229,8 +229,8 @@ async def _start_server(self):
             await self.server.serve_forever()
         except asyncio.CancelledError:
             raise
-        except OSError:
-            self._log_bind_error()
+        except OSError as e:
+            self._log_bind_error(e)
             return False
         except Exception as e:
             logging.error(e)