datalad / datalad-catalog

Create a user-friendly data catalog from structured metadata
https://datalad-catalog.netlify.app
MIT License
14 stars 12 forks source link

`set-super` does not verify validity of arguments #318

Closed mih closed 12 months ago

mih commented 1 year ago

I accidentally set id and version to something random, and it let me do that without complaint, but gave a 404 on access (unsurprisingly then).

jsheunis commented 1 year ago

Thanks for highlighting this. There's functionality that needs to be implemented (as part of the goal to implement multiple new and refactored datalad-catalog commands, see https://github.com/datalad/datalad-catalog/issues/245#issuecomment-1477051454) to report on the existence (or not) of a particular metadata record with id and version in a catalog. This would be a useful place to make use of that functionality.

jsheunis commented 12 months ago

Closed by https://github.com/datalad/datalad-catalog/pull/309