Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting upload state to Queued fails when using ingest uploads from the UI #4406

Open
tonytw1 opened this issue Jan 25, 2025 · 2 comments
Open

Comments

@tonytw1
Copy link
Contributor

tonytw1 commented Jan 25, 2025

The POST from Kahuna to move a Prepared upload into the Queued status always returns HTTP 400.

Image Image

I think this because the conditional update clause expects the status field of the upload status Dynamo table to contain the string Prepared.

(

uploadStatusTable.when(attributeExists("id") and ("status" === Prepared.toString))
)

But the StatusType enum is actually persisted in a nested format:

Image

Therefore; the query never matches and uploads never enter the Queued status.
This should show up as the 'queued' status never appearing on the upload status list view and HTTP 400s been visible in the Developer Console will uploading.

@tonytw1
Copy link
Contributor Author

tonytw1 commented Jan 25, 2025

I've bypassed this by defining a custom format to persist StatusType as StatusType.name but would be interest to know if it effects the Guardian's install.

@andrew-nowak
Copy link
Member

I've bypassed this by defining a custom format to persist StatusType as StatusType.name but would be interest to know if it effects the Guardian's install.

Gave a quick test, we do also get this 400; I'm not sure if it's new or always been the case. Either way it doesn't appear to impact processing or be visible to users, but would be great to fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants