common-workflow-language / cwltool

Common Workflow Language reference implementation
https://cwltool.readthedocs.io/
Apache License 2.0
332 stars 230 forks source link

Bad URN-scheme for `urn:hash::sha1:` #1806

Open MBueschelberger opened 1 year ago

MBueschelberger commented 1 year ago

The uri for aggregates in the provenance graph is produced by this hardcoded prefix.

According to the generic uri-scheme, double colons are not compliant to the urn-syntax.

Is there any reason for the double colon? If this was coded by accident, I would like to request a change to urn:hash:sha1: as prefix.

mr-c commented 1 year ago

Tagging @stain

Appears to have originated in https://github.com/common-workflow-language/cwltool/commit/8bc016d7a8a4a08d0782c8873adfaafbac1a4410