Skip to content

Commit

Permalink
Update log, Add flush
Browse files Browse the repository at this point in the history
This will be released as a hotfix, since symbols cannot be uploaded without the flush.
  • Loading branch information
jameskr97 committed Sep 19, 2021
1 parent a4bb480 commit 654d189
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion crashserver/webapp/operations.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ def symbol_upload(session, project_id: str, symbol_file: bytes, symbol_data: Sym
session.add(build)

if build.symbol:
logger.error("Symbol rejected. Symbol already uploaded.")
logger.error("Symbol {} already uploaded. Subsequent upload rejected.", symbol_data.build_id)
return {"error": "Symbol file already uploaded"}, 203

build.symbol = Symbol(
Expand All @@ -58,6 +58,7 @@ def symbol_upload(session, project_id: str, symbol_file: bytes, symbol_data: Sym
arch=symbol_data.arch,
app_version=symbol_data.app_version,
)
session.flush()
build.symbol.store_file(symbol_file)
session.commit()

Expand Down

0 comments on commit 654d189

Please sign in to comment.