On Wednesday, September 12, 2012 9:36:29 AM UTC-7, Jon Leighton wrote:
>
> On 12/09/12 17:29, Masterleep wrote: 
> > It could be implemented in Rails by using a container class to hold the 
> > database field names that are used as the keys inside the AR @attributes 
> > hash, and reusing the same string object across instances.
>
> To be honest I think we should just change @attributes to be keyed by 
> symbols. I don't see that there is a DoS vector in doing this since the 
> keys aren't going to come from user input (however, I do need to think 
> about that a bit more before I say so confidently). 
>


That would be even better, if it's not too hard to change to symbols. 

-- 
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To view this discussion on the web visit 
https://groups.google.com/d/msg/rubyonrails-core/-/igr8EY4pO30J.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en.

Reply via email to