search

eustasy / Phoenix

A lightweight BitTorrent Tracker written in PHP with an SQL backend.
MIT License
18 stars 6 forks source link

Added downloads counts #11

Closed enebe-nb closed 9 years ago

enebe-nb commented 9 years ago

Related issue: #10 Added downloads count as asked. Also added protection in 'info-hash' and 'peer_id' from sql injection (a 40 characters long info_hash was always concatenated without any filters). To check for vulnerabilities in another variables is recommended. 'name' column in 'torrents' table was disabled until you choose remove 'NOT NULL' or add 'DEFAULT'

lewisgoddard commented 9 years ago

This seems good, but I'll need to do some thorough testing.

lewisgoddard commented 9 years ago

Thank you for your contribution. Are you aware of the bounty on this bug? https://www.bountysource.com/issues/8628820-feature-add-downloads-totals

lewisgoddard commented 9 years ago

I meant to ask before, how did you come across this issue and what lead you to make a fix?

enebe-nb commented 9 years ago

Through bountysource, the issue looked easy and i gave a try.