Update lib/Plack/Middleware/CSRFBlock.pm#5
Update lib/Plack/Middleware/CSRFBlock.pm#5mattp- wants to merge 1 commit intorintaro:masterfrom mattp-:patch-1
Conversation
$env should be passed to token_not_found consistently, except it is not on form POST.
|
@mattp, hey, it looks like @rintaro is not really responding to any PR's here, but I've got a fork that I've been doing updates to (and have a PR in here as well). I was going to pull your PR into my repo, but after looking at it, it looks like the real bug is that he passes I think it should just be removed in the other places. If you're using this module and interested in making it better, I'm interested in that as well and may try to do a refactor in my branch when I get some time to make the code a little cleaner/more robust (I'm also running across some unicode issues with it I believe that I haven't had time to look into). Just wanted to give you a heads up in case you were interested. Thanks. |
|
@mattp, just realized he is using |
$env should be passed to token_not_found consistently, except it is not on form POST.