search

chop-dbhi / cilantro

Official client for Harvest (http://harvest.research.chop.edu)
http://cilantro.harvest.io
Other
28 stars 8 forks source link

Include csrf with Cilantro #770

Closed sgithens closed 8 years ago

sgithens commented 8 years ago

It might be nice to bundle csrf.js with Cilantro natively rather than including separately (as an example is done in verify, which is where I looked to, in order to incorporate it in my own project).

https://github.com/chop-dbhi/varify/blob/9094abbe86fc2b6260720711baa66c6e124b7ab7/varify/static/js/src/csrf.js

A simple configuration option for Cilantro to enable it would then be nice.

I'm suggesting this for inclusion because I feel like it would be necessary for a vast majority of Django projects where harvest is complementing other functionality in someones existing application.

bruth commented 8 years ago

@sgithens A long time coming, but #779 adds support for this.

sgithens commented 8 years ago

That's great! Thanks @bruth