You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the documentation it's mentioned that Storage Roots must contain a protocol, but this is not enforced. For local file storage in a local registry many users might not include file:// at the beginning, so we probably should enforce this.
The text was updated successfully, but these errors were encountered:
Use a simple regex to allow anything of the appropriate form, e.g. superstore://.
Have a predefined list of allowed protocols. If we go down this route, we should probably switch our custom URIField to Django's URLField but extend the allowed list of validators to at least include file:// (see https://docs.djangoproject.com/en/2.2/ref/validators/#urlvalidator)
I think this is a good idea, but something to implement after we submit along with the slight change to the schema (#124). We need to make sure that we can continue to make updated plots until Monday night :). And I'd go with 1 in the first instance and we can get clever later if we decide it's a good idea...
In the documentation it's mentioned that Storage Roots must contain a protocol, but this is not enforced. For local file storage in a local registry many users might not include
file://
at the beginning, so we probably should enforce this.The text was updated successfully, but these errors were encountered: