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
Is your feature request related to a problem? Please describe.
Currently we allow users to specify anything as a plugin name. We should probably validate plugin names against an allowed list of characters, as in the future with tabular output the parsing may be effected by POSIX character classes appearing within plugin names.
Agreed, as much as I like emojis, asdf-:rocket::champagne::tada: is probably not a great plugin name :joy:
So I guess the regexp would be something along the lines of asdf-[a-zA-Z0-9-] ?
Do we want to allow ., _, $ or other symbols. I kind of feel dashes are enough and we currently don't have any plugin using such symbols.
Is your feature request related to a problem? Please describe.
Currently we allow users to specify anything as a plugin name. We should probably validate plugin names against an allowed list of characters, as in the future with tabular output the parsing may be effected by POSIX character classes appearing within plugin names.
Describe the solution you'd like
Simple regex validation of plugin name in
lib/commands/command-plugin-add.bash
Additional context
[:space:]
is probably already one character we should not allowThe text was updated successfully, but these errors were encountered: