Skip to content
This repository has been archived by the owner. It is now read-only.

Set the X-CSRF-Token header for AJAX requests with jQuery. #172

Closed

Conversation

Projects
None yet
3 participants
@EspadaV8
Copy link
Member

commented Mar 25, 2012

jQuery AJAX calls weren't setting the CSRF value in the header. Currently this shouldn't be a problem but when jQuery replaces Prototype it will need to be included.

Remove issue - https://www.chiliproject.org/issues/950

@meineerde

This comment has been minimized.

Copy link
Member

commented Mar 25, 2012

Once we move to Rails 3.x, I'd like to use jquery-rails which already brings that functionality. In order to save ourselves from upgrade hassle, could you at least put a big honking TODO comment at the top of the function?

Originally I thought whether this could justify a new file that could then be removed altogether once we upgraded. But now I'm not sure anymore if that justifies the added request / latency...

@thegcat

This comment has been minimized.

Copy link
Member

commented May 20, 2012

Pushed in 61c0077, thanks!

@thegcat thegcat closed this May 20, 2012

marutosi pushed a commit to marutosi/chili that referenced this pull request Jun 17, 2013

Merge pull request chiliproject#172 from opf/feature/rails3_remove_bu…
…ndled_acts_as_list

Feature/rails3 remove bundled acts as list
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.