Skip to content

Commit

Permalink
Fix fastapi syntax for optional params
Browse files Browse the repository at this point in the history
  • Loading branch information
deajan committed Nov 20, 2023
1 parent ac76aa7 commit dac0dce
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions grafana_webhook_gammu_smsd/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
__appname__ = "Grafana 2 Gammu SMSD"


from typing import Optional
from typing import Union
from command_runner import command_runner
import logging
import secrets
Expand Down Expand Up @@ -94,7 +94,7 @@ async def api_root(auth=Depends(get_current_username)):

@app.post("/grafana/{numbers}")
@app.post("/grafana/{numbers}/{min_interval}")
async def grafana(numbers: str, min_interval: Optional[int], alert: AlertMessage, auth=Depends(auth_scheme)):
async def grafana(numbers: str, min_interval: Union[int, None] = None, alert: AlertMessage = None, auth=Depends(auth_scheme)):

global LAST_SENT_TIMESTAMP

Expand All @@ -104,10 +104,10 @@ async def grafana(numbers: str, min_interval: Optional[int], alert: AlertMessage
detail="No phone number set"
)

if not alert.message:
if not alert or not alert.message:
raise HTTPException(
stats_code=404,
detial="No alert set"
status_code=404,
detail="No alert set"
)

# Multiple numbers with ';' are accepted
Expand Down

0 comments on commit dac0dce

Please sign in to comment.