-
Notifications
You must be signed in to change notification settings - Fork 240
Image "Name" Regex Nit #484
Copy link
Copy link
Closed
Description
Currently the repo name regex in the spec is,
[a-z0-9]+((\.|_|__|-+)[a-z0-9]+)*(/[a-z0-9]+((\.|_|__|-+)[a-z0-9]+)*)*
Some regex parsers want / to be escaped, however others do not recognize an escaped /. I propose using [/] to avoid ambiguity.
[a-z0-9]+((\.|_|__|-+)[a-z0-9]+)*([/][a-z0-9]+((\.|_|__|-+)[a-z0-9]+)*)*
^^^ ---- change is hereAnother minor thing I noticed that I'm piggybacking on this issue. Why are capital letters unallowed for the repo name, but allowed for the tag?
Edit
Example of a parser that will warn about escaping,
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels