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

Order takes one argument, 'allow,deny', 'deny,allow', or 'mutual-failure... #5197

Closed
wants to merge 1 commit into from
Closed

Conversation

fpondepeyre
Copy link

Q A
Doc fix? yes
New docs? no

Syntax error : order takes one argument, 'allow,deny', 'deny,allow', or 'mutual-failure'

…ure'

Syntax error : order takes one argument, 'allow,deny', 'deny,allow', or 'mutual-failure'
@xabbuh
Copy link
Member

xabbuh commented Apr 21, 2015

👍 good catch @fpondepeyre

@fpondepeyre fpondepeyre changed the title order takes one argument, 'allow,deny', 'deny,allow', or 'mutual-failure... Order takes one argument, 'allow,deny', 'deny,allow', or 'mutual-failure... Apr 21, 2015
@xabbuh
Copy link
Member

xabbuh commented Apr 23, 2015

@fpondepeyre In #5203 (comment) it was suggested to uppercase the option value to.Will you be able to make this change?

@xabbuh
Copy link
Member

xabbuh commented May 6, 2015

ping @fpondepeyre :)

@wouterj
Copy link
Member

wouterj commented May 10, 2015

I'm closing this PR in favor of #5203. It was just a little more complete and because we get a lot of bug fixes about this problem, I want to fix this bug as quick as possible.

Anyway, thanks for noticing and fixing this bug @fpondepeyre!

@wouterj wouterj closed this May 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants