Bug 56235 - quotes should be optional around expr
Summary: quotes should be optional around expr
Status: RESOLVED FIXED
Alias: None
Product: Apache httpd-2
Classification: Unclassified
Component: mod_authz_core (show other bugs)
Version: 2.5-HEAD
Hardware: PC Linux
: P2 normal (vote)
Target Milestone: ---
Assignee: Apache HTTPD Bugs Mailing List
URL:
Keywords: FixedInTrunk
Depends on:
Blocks:
 
Reported: 2014-03-07 21:15 UTC by Rich Bowen
Modified: 2016-12-31 00:42 UTC (History)
0 users


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Rich Bowen 2014-03-07 21:15:11 UTC 
Example from expr.html:

Require expr "%{TIME_HOUR} -gt 9 && %{TIME_HOUR} -lt 17"

fails with "unexpected $end"

while

Require expr %{TIME_HOUR} -gt 9 && %{TIME_HOUR} -lt 17

(ie, with quotes removed) works as expected. (Doc has since been updated.)

It seems to me that the quotes should be (optionally) permitted around the expression, as this would be consistent with other modules which use the expression syntax.
Comment 1 Ben Reser 2014-04-07 22:39:27 UTC 
Fixed on trunk in r1585609.
Comment 2 Rich Bowen 2014-04-07 23:06:51 UTC 
Thanks. You rock!
Comment 3 Eric Covener 2016-12-31 00:42:24 UTC 
Fixed in 2.4.16