search

deleteman / lfpr

Find projects to contribute to or publish your project so others can find it.
www.lookingforpullrequests.com
64 stars 14 forks source link

fix sql injection #48

Closed gregmolnar closed 10 years ago

gregmolnar commented 10 years ago

Just 1 example of the issue and how to fix #47. But maybe an overall architecture change would be better.

deleteman commented 10 years ago

Thanks for the example, but I'll close this one since we're going with the overall architecture change.

mpscholten commented 10 years ago

@deleteman IMO we should merge this asap to fix the injection before someone is abusing it. This won't hurt our refactor of the overall system :-)