Skip to content

Commit

Permalink
fix: updated pre-commit, and fixed false positives
Browse files Browse the repository at this point in the history
  • Loading branch information
fstagni authored and chaen committed Jan 13, 2025
1 parent 002403c commit 7210549
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ repos:
- id: check-added-large-files

- repo: https://github.com/astral-sh/ruff-pre-commit
rev: 'v0.8.3'
rev: 'v0.9.1'
hooks:
- id: ruff
args: ["--fix"]
Expand All @@ -24,7 +24,7 @@ repos:
- id: black

- repo: https://github.com/pre-commit/mirrors-mypy
rev: v1.13.0
rev: v1.14.1
hooks:
- id: mypy
additional_dependencies:
Expand Down
2 changes: 1 addition & 1 deletion diracx-core/src/diracx/core/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ class TokenResponse(BaseModel):
# Based on RFC 6749
access_token: str
expires_in: int
token_type: str = "Bearer"
token_type: str = "Bearer" # noqa: S105
refresh_token: str | None = None


Expand Down
4 changes: 2 additions & 2 deletions diracx-routers/src/diracx/routers/utils/users.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,9 +61,9 @@ class AuthSettings(ServiceSettingsBase):

# TODO: this should probably be something mandatory
# to set by the user
token_issuer: str = "http://lhcbdirac.cern.ch/"
token_issuer: str = "http://lhcbdirac.cern.ch/" # noqa: S105
token_key: TokenSigningKey
token_algorithm: str = "RS256"
token_algorithm: str = "RS256" # noqa: S105
access_token_expire_minutes: int = 20
refresh_token_expire_minutes: int = 60

Expand Down

0 comments on commit 7210549

Please sign in to comment.