Any word on the reset_session bug in trunk? - I attached a new test as per Rick Olson's request showing the failure. Here it is again (diff of session_management_test.rb). Let me know if there is anything else I can do.
Thanks, Zack On 7/17/06, Daniel N <[EMAIL PROTECTED]> wrote:
On 7/15/06, Rick Olson <[EMAIL PROTECTED]> wrote: > On 7/14/06, Zack Chandler <[EMAIL PROTECTED]> wrote: > > If you comment out the reset_session line the code works fine. > > Looks like a bug to me. I always like to reset the session upon login > > to protect against hijacked sessions. > > > > Should I submit a bug report when trac comes back to life? > > > > Zack > > If you could provide a failing action controller test, that'd be great > :) I'll try to get a look at this sometime this weekend if someone > else doesn't beat me to it. I'd say just throw a diff somewhere or > even attach it to one of these mails. Hi, I'm guessing this one isn't sorted yet. I just got edge up to date (rev 4610) and I have the same error on two tests in the user_controller for AAA test_should_logout test_should_delete_token_on_logout I went back to rails 1.1.4 and no problem. Any news on this one? Cheers Dan > -- Rick Olson http://techno-weenie.net _______________________________________________ Rails-core mailing list Rails-core@lists.rubyonrails.org http://lists.rubyonrails.org/mailman/listinfo/rails-core _______________________________________________ Rails-core mailing list Rails-core@lists.rubyonrails.org http://lists.rubyonrails.org/mailman/listinfo/rails-core
45a46,54 > > class BasicController < ActionController::Base > > def reset > reset_session > render :text => 'reset' > end > > end 51a61,65 > def test_reset_session > @controller = BasicController.new > get :reset > end >
_______________________________________________ Rails-core mailing list Rails-core@lists.rubyonrails.org http://lists.rubyonrails.org/mailman/listinfo/rails-core
