spree icon indicating copy to clipboard operation
spree copied to clipboard

[WIP] SQL injection in Products::Find causing `undefined method distinct for nil:NilClass`

Open mbajur opened this issue 1 year ago • 2 comments

Recently, an attacker attempted to inject an SQL statement via the sort_by param for TaxonsController#show on my shop. Even though the attack didn't succeeded, it produced several thousand exception reports for:

image

This PR aims to make sure given scenario does not produce 500 error.

mbajur avatar Dec 29 '24 13:12 mbajur

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

github-actions[bot] avatar Dec 29 '24 13:12 github-actions[bot]

I have read the CLA Document and I hereby sign the CLA

mbajur avatar Dec 29 '24 13:12 mbajur