-
Notifications
You must be signed in to change notification settings - Fork 35
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
Support multipart? #1
Comments
This fixes it for me: It uses the values from revel ParamsFilter which has already been executed. |
Hi Otto. Thanks for taking time to fix the issue. I've updated the GitHub On Mon, Sep 9, 2013 at 4:28 PM, Otto Bretz notifications@github.com wrote:
|
I think my pull request has overwritten this fix... Please fix it again. |
I'm new to Github. Learning curve is difficult... Should be fixed now. On Wed, Sep 11, 2013 at 9:41 AM, Allen Dang notifications@github.comwrote:
|
Thanks for this filter, it's working great but I have found one small issue. I have a form with
enctype="multipart/form-data"
, my guess is that this causes problems withsentToken = r.PostFormValue(fieldName)
. In this case my sentToken becomes an empty string and the csrf check fails.The text was updated successfully, but these errors were encountered: