Add additional arguments to decorator
This allows for `@requires_team_membership()` to be able to retrieve the team_id from a JSON or query parameter.master
parent
3ff1ef7683
commit
061499b822
|
@ -72,7 +72,7 @@ class CreateEventJson(BaseModel):
|
|||
)
|
||||
)
|
||||
@requires_authentication
|
||||
@requires_team_membership
|
||||
@requires_team_membership()
|
||||
def create_event(player_team: PlayerTeam, json: CreateEventJson, **_):
|
||||
event = Event()
|
||||
event.team_id = player_team.team_id
|
||||
|
@ -107,7 +107,7 @@ def create_event(player_team: PlayerTeam, json: CreateEventJson, **_):
|
|||
|
||||
@api_events.patch("/<int:event_id>/players")
|
||||
@requires_authentication
|
||||
@requires_team_membership
|
||||
@requires_team_membership()
|
||||
def set_event_players(player_team: PlayerTeam, event_id: int, **_):
|
||||
assert_team_authority(player_team, None)
|
||||
|
||||
|
|
|
@ -1,5 +1,7 @@
|
|||
from functools import wraps
|
||||
from typing import Optional
|
||||
from flask import abort, make_response, request
|
||||
from sqlalchemy.sql.operators import json_path_getitem_op
|
||||
from app_db import db
|
||||
from models.auth_session import AuthSession
|
||||
from models.player import Player
|
||||
|
@ -28,15 +30,32 @@ def requires_authentication(f):
|
|||
return f(*args, **kwargs)
|
||||
return decorator
|
||||
|
||||
def requires_team_membership(f):
|
||||
def requires_team_membership(
|
||||
path_param: Optional[str] = None,
|
||||
json_param: Optional[str] = None,
|
||||
query_param: Optional[str] = None
|
||||
):
|
||||
def wrapper(f):
|
||||
@wraps(f)
|
||||
def decorator(*args, **kwargs):
|
||||
player: Player | None = kwargs["player"]
|
||||
team_id: int = kwargs["team_id"]
|
||||
|
||||
team_id: int
|
||||
if path_param:
|
||||
team_id = kwargs[path_param]
|
||||
elif json_param:
|
||||
team_id = getattr(kwargs["json"], json_param)
|
||||
elif query_param:
|
||||
team_id = getattr(kwargs["query"], query_param)
|
||||
else:
|
||||
team_id = kwargs["team_id"]
|
||||
|
||||
if not player:
|
||||
abort(401)
|
||||
|
||||
if not team_id:
|
||||
abort(500)
|
||||
|
||||
player_team = db.session.query(
|
||||
PlayerTeam
|
||||
).where(
|
||||
|
@ -51,6 +70,7 @@ def requires_team_membership(f):
|
|||
kwargs["player_team"] = player_team
|
||||
return f(*args, **kwargs)
|
||||
return decorator
|
||||
return wrapper
|
||||
|
||||
def assert_team_authority(
|
||||
player_team: PlayerTeam,
|
||||
|
|
|
@ -55,7 +55,7 @@ def get_integrations(player: Player, team_id: int, **_):
|
|||
operation_id="create_integration"
|
||||
)
|
||||
@requires_authentication
|
||||
@requires_team_membership
|
||||
@requires_team_membership()
|
||||
def create_integration(player_team: PlayerTeam, integration_type: str, **_):
|
||||
assert_team_authority(player_team)
|
||||
|
||||
|
@ -81,7 +81,7 @@ def create_integration(player_team: PlayerTeam, integration_type: str, **_):
|
|||
operation_id="delete_integration"
|
||||
)
|
||||
@requires_authentication
|
||||
@requires_team_membership
|
||||
@requires_team_membership()
|
||||
def delete_integration(player_team: PlayerTeam, integration_id: int, **_):
|
||||
assert_team_authority(player_team)
|
||||
|
||||
|
@ -109,7 +109,7 @@ def delete_integration(player_team: PlayerTeam, integration_id: int, **_):
|
|||
operation_id="update_integration"
|
||||
)
|
||||
@requires_authentication
|
||||
@requires_team_membership
|
||||
@requires_team_membership()
|
||||
def update_integration(
|
||||
player_team: PlayerTeam,
|
||||
integration_id: int,
|
||||
|
|
|
@ -22,7 +22,7 @@ api_team_invite = Blueprint("team_invite", __name__)
|
|||
operation_id="get_invites"
|
||||
)
|
||||
@requires_authentication
|
||||
@requires_team_membership
|
||||
@requires_team_membership()
|
||||
def get_invites(team_id: int, **_):
|
||||
invites = db.session.query(
|
||||
TeamInvite
|
||||
|
@ -48,7 +48,7 @@ def get_invites(team_id: int, **_):
|
|||
operation_id="create_invite"
|
||||
)
|
||||
@requires_authentication
|
||||
@requires_team_membership
|
||||
@requires_team_membership()
|
||||
def create_invite(team_id: int, **_):
|
||||
team_id_shifted = int(team_id) << 48
|
||||
random_value_shifted = int(randint(0, (1 << 16) - 1)) << 32
|
||||
|
|
Loading…
Reference in New Issue