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.

Sign up for GitHub

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

Jump to bottom

Allow periods in the variable name #287

Merged

rodneyrehm merged 1 commit into medialize:master from keanu-delgado:patch-1

Mar 24, 2016

Contributor

keanu-delgado commented Mar 24, 2016

From https://tools.ietf.org/html/rfc6570#section-2.3 :

variable-list =  varspec *( "," varspec )
varspec       =  varname [ modifier-level4 ]
varname       =  varchar *( ["."] varchar )
varchar       =  ALPHA / DIGIT / "_" / pct-encoded


          Allow periods in the variable name

d11373d

Member

rodneyrehm commented Mar 24, 2016

Nice!
(My RegExp is so rusty, I spent the last 10 minutes trying to understand your change)

rodneyrehm merged commit 670ea0d into medialize:master

keanu-delgado deleted the patch-1 branch

March 25, 2016 14:15

rodneyrehm added a commit that referenced this pull request


          chore(changelog): noting #287, #289, #290

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet