Skip to content

Commit 4392ef7

Browse files
committed
linting
Signed-off-by: Keval Mahajan <[email protected]>
1 parent 7983b7a commit 4392ef7

File tree

4 files changed

+15
-5
lines changed

4 files changed

+15
-5
lines changed

mcpgateway/config.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1293,7 +1293,9 @@ def validate_database(self) -> None:
12931293
db_dir.mkdir(parents=True)
12941294

12951295
# Validation patterns for safe display (configurable)
1296-
validation_dangerous_html_pattern: str = r"<(script|iframe|object|embed|link|meta|base|form|img|svg|video|audio|source|track|area|map|canvas|applet|frame|frameset|html|head|body|style)\b|</*(script|iframe|object|embed|link|meta|base|form|img|svg|video|audio|source|track|area|map|canvas|applet|frame|frameset|html|head|body|style)>"
1296+
validation_dangerous_html_pattern: str = (
1297+
r"<(script|iframe|object|embed|link|meta|base|form|img|svg|video|audio|source|track|area|map|canvas|applet|frame|frameset|html|head|body|style)\b|</*(script|iframe|object|embed|link|meta|base|form|img|svg|video|audio|source|track|area|map|canvas|applet|frame|frameset|html|head|body|style)>"
1298+
)
12971299

12981300
validation_dangerous_js_pattern: str = r"(?i)(?:^|\s|[\"'`<>=])(javascript:|vbscript:|data:\s*[^,]*[;\s]*(javascript|vbscript)|\bon[a-z]+\s*=|<\s*script\b)"
12991301

mcpgateway/routers/auth.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,9 @@ async def login(login_request: LoginRequest, request: Request, db: Session = Dep
153153
logger.info(f"User {email} authenticated successfully")
154154

155155
# Return session token for UI access and API key management
156-
return AuthenticationResponse(access_token=access_token, token_type="bearer", expires_in=expires_in, user=EmailUserResponse.from_email_user(user)) # nosec B106 - OAuth2 token type, not a password
156+
return AuthenticationResponse(
157+
access_token=access_token, token_type="bearer", expires_in=expires_in, user=EmailUserResponse.from_email_user(user)
158+
) # nosec B106 - OAuth2 token type, not a password
157159

158160
except ValueError as e:
159161
logger.warning(f"Login validation error: {e}")

mcpgateway/routers/email_auth.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -251,7 +251,9 @@ async def login(login_request: EmailLoginRequest, request: Request, db: Session
251251
access_token, expires_in = await create_access_token(user)
252252

253253
# Return authentication response
254-
return AuthenticationResponse(access_token=access_token, token_type="bearer", expires_in=expires_in, user=EmailUserResponse.from_email_user(user)) # nosec B106 - OAuth2 token type, not a password
254+
return AuthenticationResponse(
255+
access_token=access_token, token_type="bearer", expires_in=expires_in, user=EmailUserResponse.from_email_user(user)
256+
) # nosec B106 - OAuth2 token type, not a password
255257

256258
except Exception as e:
257259
logger.error(f"Login error for {login_request.email}: {e}")
@@ -300,7 +302,9 @@ async def register(registration_request: EmailRegistrationRequest, request: Requ
300302

301303
logger.info(f"New user registered: {user.email}")
302304

303-
return AuthenticationResponse(access_token=access_token, token_type="bearer", expires_in=expires_in, user=EmailUserResponse.from_email_user(user)) # nosec B106 - OAuth2 token type, not a password
305+
return AuthenticationResponse(
306+
access_token=access_token, token_type="bearer", expires_in=expires_in, user=EmailUserResponse.from_email_user(user)
307+
) # nosec B106 - OAuth2 token type, not a password
304308

305309
except EmailValidationError as e:
306310
raise HTTPException(status_code=status.HTTP_400_BAD_REQUEST, detail=str(e))

mcpgateway/services/gateway_service.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1780,7 +1780,9 @@ async def delete_gateway(self, db: Session, gateway_id: str, user_email: Optiona
17801780
db.rollback()
17811781
raise GatewayError(f"Failed to delete gateway: {str(e)}")
17821782

1783-
async def forward_request(self, gateway_or_db, method: str, params: Optional[Dict[str, Any]] = None, app_user_email: Optional[str] = None) -> Any: # noqa: F811 # pylint: disable=function-redefined
1783+
async def forward_request(
1784+
self, gateway_or_db, method: str, params: Optional[Dict[str, Any]] = None, app_user_email: Optional[str] = None
1785+
) -> Any: # noqa: F811 # pylint: disable=function-redefined
17841786
"""
17851787
Forward a request to a gateway or multiple gateways.
17861788

0 commit comments

Comments
 (0)