This was too restrictive, making ENV flag essentially useless (there
is no way to access rewrite capture groups or other environment
variables).
This still forbids spaces; we could allow them, but would have to
ensure proper quoting or escaping and the apache parser for rewrite
arguments is pretty funky, so this seems to be of dubious value (read:
it would be a lot more work).
See https://bugzilla.hcoop.net/show_bug.cgi?id=1287 for background.
val _ = Env.type_one "rewrite_arg"
Env.string
- (CharVector.all Char.isAlphaNum)
+ (CharVector.all (fn ch => (Char.isGraph ch) andalso not (List.exists (fn c => ch = c) [ #"[", #"]", #",", #"\"", #"'", #"=", #":" ])))
val _ = Env.type_one "suexec_flag"
Env.bool