Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Httpd: can't parse RewriteCond with empty string in conditional #429

Closed
domcleal opened this issue Dec 1, 2016 · 0 comments
Closed

Httpd: can't parse RewriteCond with empty string in conditional #429

domcleal opened this issue Dec 1, 2016 · 0 comments

Comments

@domcleal
Copy link
Member

domcleal commented Dec 1, 2016

The following valid Apache syntax fails to parse with the Httpd module:

RewriteCond ${movedPageMap:$1}  >""

Adding characters into the quoted string at the end (a greater-than comparison) parses successfully, so it appears to only be an issue with the empty quoted string.

(Originally reported at certbot/certbot#3536)

lutter added a commit to lutter/augeas that referenced this issue Sep 27, 2017
* handle '>""' properly
* make space between quoted args optional
* do not get confused by quoted strings in bare arguments

Fixes hercules-team#429
Fixes hercules-team#435
Fixes hercules-team#470
lutter added a commit to lutter/augeas that referenced this issue Sep 27, 2017
* handle '>""' properly
* make space between quoted args optional
* do not get confused by quoted strings in bare arguments

Fixes hercules-team#429
Fixes hercules-team#435
Fixes hercules-team#470
lutter added a commit to lutter/augeas that referenced this issue Sep 27, 2017
* handle '>""' properly
* make space between quoted args optional
* do not get confused by quoted strings in bare arguments

Fixes hercules-team#429
Fixes hercules-team#435
Fixes hercules-team#470
lutter added a commit to lutter/augeas that referenced this issue Sep 29, 2017
* handle '>""' properly
* make space between quoted args optional
* do not get confused by quoted strings in bare arguments

Fixes hercules-team#429
Fixes hercules-team#435
Fixes hercules-team#470
@lutter lutter closed this as completed in e1a8608 Sep 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant