diff options
author | Bryan Newbold <bnewbold@archive.org> | 2021-01-15 01:17:32 -0800 |
---|---|---|
committer | Bryan Newbold <bnewbold@archive.org> | 2021-01-15 01:17:32 -0800 |
commit | 78472e82535da109f63e1b5c5ce8465016c2f8cc (patch) | |
tree | 7a5c5d7e76a12d7fdd664feb3e8a1f57b27ab1fa /fatcat_scholar/web.py | |
parent | 01dc3421dca9afc33af4effdd857b33b20d9a6c9 (diff) | |
download | fatcat-scholar-78472e82535da109f63e1b5c5ce8465016c2f8cc.tar.gz fatcat-scholar-78472e82535da109f63e1b5c5ce8465016c2f8cc.zip |
api: fix /search test, and mypy error on implementation
Diffstat (limited to 'fatcat_scholar/web.py')
-rw-r--r-- | fatcat_scholar/web.py | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/fatcat_scholar/web.py b/fatcat_scholar/web.py index 0b82df9..fe2ad90 100644 --- a/fatcat_scholar/web.py +++ b/fatcat_scholar/web.py @@ -9,7 +9,7 @@ from typing import Optional, Any, List from pydantic import BaseModel import babel.support -from fastapi import FastAPI, APIRouter, Request, Depends, Response +from fastapi import FastAPI, APIRouter, Request, Depends, Response, HTTPException from fastapi.staticfiles import StaticFiles from fastapi.responses import PlainTextResponse, JSONResponse, FileResponse import sentry_sdk @@ -93,16 +93,16 @@ class HitsModel(BaseModel): @api.get("/search", operation_id="get_search", response_model=HitsModel) async def search(query: FulltextQuery = Depends(FulltextQuery)) -> FulltextHits: - if query.q is not None: - try: - hits: FulltextHits = do_fulltext_search(query) - except ValueError as e: - sentry_sdk.set_level("warning") - sentry_sdk.capture_exception(e) - raise HTTPException(status_code=400, detail=f"Query Error: {e}") - except IOError as e: - sentry_sdk.capture_exception(e) - raise HTTPException(status_code=500, detail=f"Backend Error: {e}") + hits: Optional[FulltextHits] = None + try: + hits = do_fulltext_search(query) + except ValueError as e: + sentry_sdk.set_level("warning") + sentry_sdk.capture_exception(e) + raise HTTPException(status_code=400, detail=f"Query Error: {e}") + except IOError as e: + sentry_sdk.capture_exception(e) + raise HTTPException(status_code=500, detail=f"Backend Error: {e}") # remove internal context from hit objects for doc in hits.results: |