Browse Source

Ban @ from xcase__enum()

Turns out that case ids may end up being re-used in URI contexts, and
'@' has special meaning in some parts of URL (HTTP, at least), which
could cause confusion or make translations to HTTP URLs troublesome.

Since it looks like it's not used anywhere yet, it's safer to drop it.
Alois Mahdal 6 years ago
parent
commit
f4d117cf8e
1 changed files with 1 additions and 1 deletions
  1. 1
    1
      src/xcase.sh.skel

+ 1
- 1
src/xcase.sh.skel View File

@@ -1135,7 +1135,7 @@ __xcase__validate_enum() {
1135 1135
     #
1136 1136
     # Make sure stuff from xcase__enum() has no banned chars
1137 1137
     #
1138
-    local allowed='[:alnum:]._,+%@=-'   # allowed chars in case id
1138
+    local allowed='[:alnum:]._,+%=-'    # allowed chars in case id
1139 1139
     local es=                           # exit status of this function
1140 1140
     if grep "[^$allowed]" "$__xcase__tmp/enum";
1141 1141
     then