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

Verify behaviour with newer param access methods #4

Open
jbarrett opened this issue Nov 17, 2015 · 3 comments
Open

Verify behaviour with newer param access methods #4

jbarrett opened this issue Nov 17, 2015 · 3 comments
Assignees
Labels

Comments

@jbarrett
Copy link
Owner

No description provided.

@SysPete
Copy link
Contributor

SysPete commented Jul 29, 2016

As noted in PerlDancer/Dancer2#1192 the new parameters methods were not decoding their inputs so UnicodeNormalize would have no chance of being able to do anything sane. I've created a PR for Dancer2: PerlDancer/Dancer2#1219

Expect a PR for this plugin really soon (though it will of course have to wait for Dancer2 #1219 to be merged).

@jbarrett
Copy link
Owner Author

Many thanks @SysPete!

I was leaning towards trying to make this a Plack middleware and having the plugin add it to the stack. Not too sure how feasible that is either. Low on tuits, so haven't experimented much.

@jbarrett
Copy link
Owner Author

For what it's worth, I think I was able to apply normalization to all parameters except body, but this could be faulty Friday-night memory talking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants