search

ring-clojure / ring-anti-forgery

Ring middleware to prevent CSRF attacks
119 stars 26 forks source link

Makes the anti-forgery token available to handlers from the request #19

Closed timothypratley closed 6 years ago

timothypratley commented 6 years ago

See discussion here: https://github.com/ring-clojure/ring-defaults/issues/23

weavejester commented 6 years ago

Also please follow the seven rules of a great git commit message, as mentioned in the Ring contributing guidelines.

weavejester commented 6 years ago

Oh, and thanks for the PR :)

timothypratley commented 6 years ago

My pleasure :) Thank you for the guidance. I have made the changes you identified. Let me know if anything remains amiss.

weavejester commented 6 years ago

Thanks! Can you change the commit message to:

Add :anti-forgery-token to the request map

Provides an alternate route for handlers to get the anti-forgery token.
See: ring-clojure/ring-defaults#23
timothypratley commented 6 years ago

sure, done :)

weavejester commented 6 years ago

Thanks! I just have one minor naming comment to address, and then I'll merge it in and cut a release.

timothypratley commented 6 years ago

good catch; updated.