fix: set default log level to ERROR (#73)

Author: wzxxing

mcp_proxy_for_aws/cli.py

@@ -73,8 +73,8 @@ def parse_args():
     parser.add_argument(
         '--log-level',
         choices=['DEBUG', 'INFO', 'WARNING', 'ERROR', 'CRITICAL'],
-        default='INFO',
-        help='Set the logging level (default: INFO)',
+        default='ERROR',
+        help='Set the logging level (default: ERROR)',
     )
 
     parser.add_argument(

mcp_proxy_for_aws/sigv4_helper.py

@@ -84,27 +84,41 @@ async def _handle_error_response(response: httpx.Response) -> None:
         No raises. let the mcp http client handle the errors.
     """
     if response.is_error:
+        # warning only because the SDK logs error
+        log_level = logging.WARNING
+        if (
+            # The server MAY respond 405 to GET (SSE) and DELETE (session).
+            response.status_code == 405 and response.request.method in ('GET', 'DELETE')
+        ) or (
+            # The server MAY terminate the session at any time, after which it MUST
+            # respond to requests containing that session ID with HTTP 404 Not Found.
+            response.status_code == 404 and response.request.method == 'POST'
+        ):
+            log_level = logging.DEBUG
+
         try:
             # read the content and settle the response content. required to get body (.json(), .text)
             await response.aread()
         except Exception as e:
-            logger.error('Failed to read response: %s', e)
-            # do nothing and let the client handle the error
+            logger.debug('Failed to read response: %s', e)
+            # do nothing and let the client and SDK handle the error
             return
 
         # Try to extract error details with fallbacks
         try:
             # Try to parse JSON error details
             error_details = response.json()
-            logger.error('HTTP %d Error Details: %s', response.status_code, error_details)
+            logger.log(log_level, 'HTTP %d Error Details: %s', response.status_code, error_details)
         except Exception:
             # If JSON parsing fails, use response text or status code
             try:
                 response_text = response.text
-                logger.error('HTTP %d Error: %s', response.status_code, response_text)
+                logger.log(log_level, 'HTTP %d Error: %s', response.status_code, response_text)
             except Exception:
                 # Fallback to just status code and URL
-                logger.error('HTTP %d Error for url %s', response.status_code, response.url)
+                logger.log(
+                    log_level, 'HTTP %d Error for url %s', response.status_code, response.url
+                )
 
 
 def create_aws_session(profile: Optional[str] = None) -> boto3.Session:

tests/unit/test_cli.py

@@ -32,7 +32,7 @@ def test_parse_args_minimal(self):
         assert args.profile is None
         assert args.region is None
         assert args.read_only is False
-        assert args.log_level == 'INFO'
+        assert args.log_level == 'ERROR'
         assert args.retries == 0
         assert args.timeout == 180.0
         assert args.connect_timeout == 60.0

tests/unit/test_server.py

@@ -194,7 +194,7 @@ def test_parse_args_default(self):
         assert args.region is None
         assert args.profile is None
         assert args.read_only is False
-        assert args.log_level == 'INFO'
+        assert args.log_level == 'ERROR'
         assert args.retries == 0
 
     @patch(