Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

[PATCH] Bug: Earlier Check for Session in Forgery Protection

#139

The session is used by the form_authenticity_token method before it is

tested to be valid. This patch moves a few lines around so that the

session is validated first.

Without this patch, if you try to use forgery protection with sessions

turned off, you get this exception message:

undefined method `session_id' for {}:Hash

The patch includes a test that can be used to see this behavior before

the request_forgery_protection.rb file is patched to fix it.

Reported by Peter Jones · May 7th, 2008 @ 11:07 PM

State: resolved
Milestone: none
Assigned to: nobody
Importance: none

Activity

  1. Jim James
  2. Rafael Cardoso
  3. Fernand Galiana
  4. _ugly
  5. Alistair Holt
  6. Andrew Zielinski
  7. Repository
    Repository
    • State changed from new to resolved

    (from [0dabb5b7ab3fad23da91a2312f7b586855d52f4a]) Fixed that forgery protection can be used without session tracking (Peter Jones) [#139 state:resolved]

    http://github.com/rails/rails/co...

    May 11th, 2008 @ 07:29 PM

  8. af001
    af001
    • Tag set to actionpack, bug, edge, patch, request-forgery-protection, tested
    • Importance changed from to

    私の中で、総合評価のとっても低いアバアバクロホリスタークロ銀座店。アバクロは大好きなんですけどね。一昨日の東京駅付近での打ち合わせの後、散歩がてら久々に行ってきました。そしたらビックリ!相変わらアバクロず、

    May 5th, 2011 @ 02:54 AM