oops, omitted a couple important features/vulnerabilities
This commit is contained in:
+2
-2
@@ -62,7 +62,7 @@ class User < ActiveRecord::Base
|
|||||||
return auth
|
return auth
|
||||||
end
|
end
|
||||||
|
|
||||||
#=begin
|
=begin
|
||||||
# More secure version, still lacking a decent hashing routine, this is for timing attack prevention
|
# More secure version, still lacking a decent hashing routine, this is for timing attack prevention
|
||||||
def self.authenticate(email, password)
|
def self.authenticate(email, password)
|
||||||
user = find_by_email(email) || User.new(:password => "")
|
user = find_by_email(email) || User.new(:password => "")
|
||||||
@@ -72,7 +72,7 @@ class User < ActiveRecord::Base
|
|||||||
raise "Incorrect username or password"
|
raise "Incorrect username or password"
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
#=end
|
=end
|
||||||
|
|
||||||
def assign_user_id
|
def assign_user_id
|
||||||
unless @skip_user_id_assign.present? || self.user_id.present?
|
unless @skip_user_id_assign.present? || self.user_id.present?
|
||||||
|
|||||||
@@ -26,7 +26,7 @@
|
|||||||
going on with funny chars and jquery, plus it says safe so I'm guessing
|
going on with funny chars and jquery, plus it says safe so I'm guessing
|
||||||
nothing bad will happen
|
nothing bad will happen
|
||||||
-->
|
-->
|
||||||
Welcome, <%= current_user.first_name %>
|
Welcome, <%= current_user.first_name.html_safe %>
|
||||||
</li>
|
</li>
|
||||||
<li>
|
<li>
|
||||||
<%= button_to "RailsGoat Tutorials", tutorials_path, {:class => "btn btn-primary", :method => "get"}%>
|
<%= button_to "RailsGoat Tutorials", tutorials_path, {:class => "btn btn-primary", :method => "get"}%>
|
||||||
|
|||||||
Reference in New Issue
Block a user