pulibrary / figgy

Valkyrie-based digital repository backend.
Other
35 stars 4 forks source link

Music program is failing to mint ARKs and thus can't be completed #5860

Open tpendragon opened 1 year ago

tpendragon commented 1 year ago

The params say post but the url says get? from params it looks like an issue with changing the resource state.

Backtrace

line 41 of [PROJECT_ROOT]/app/services/identifier_service.rb: update_metadata
line 7 of [PROJECT_ROOT]/app/services/identifier_service.rb: mint_or_update
line 22 of [PROJECT_ROOT]/app/change_set_persisters/change_set_persister/mint_identifier.rb: mint_identifier

View full backtrace and more info at honeybadger.io

Sudden Priority Justification

Dan's unable to get this program up while this error persists, and we suspect it'll cause further problems if they put the ARK in first somehow.

Note: The records that were stopping work have been fixed, so we've pulled this off sudden priority.

sdellis commented 1 year ago

Example: https://figgy.princeton.edu/catalog/4b8e1559-2695-4f2b-9620-f8105a0405d9

tpendragon commented 1 year ago

I think the problem here is that the imported ARK is the PDF ARK, so it's trying to update an ARK with /pdf at the end. We should strip that on metadata import.

Imported identifier:

ark:/88435/dc1c18ds73h/pdf

hackartisan commented 1 year ago

I checked the honeybadger issue for other resources that had this error. Here are the IDs:

tpendragon commented 1 year ago

Fixed.

hackartisan commented 1 year ago

This failed again on https://figgy.princeton.edu/catalog/dec36166-c167-43a9-a858-a6ec824406d6

tpendragon commented 1 year ago

I suspect this error's a little different - someone put the wrong ark in the record as a placeholder prior to completing the record.

hackartisan commented 1 year ago

Yes, I should have mentioned that -- I reopened because I wasn't sure what to do about it! Does it just not require action?

hackartisan commented 1 year ago

Figgy is trying to update the ark in ezid but the ark isn't valid, so it's failing. Things we could do:

How can we tell someone to fix this?

tpendragon commented 1 year ago

Fix it with this: don't propagate arks from Orangelight that don't have ark:// in them (don't pull them in from alma record)