Skip to content

Image "Name" Regex Nit #484

@juliusl

Description

@juliusl

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 here

Another 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,

https://regexr.com/7lrpf

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions