We're updating the issue view to help you get more done.Learn more

Abuse reports should get IP address in controller, LiveValidation should work on summary field

Abuse reports include IP addresses. Right now, we do so via a hidden field on the form, which makes it easier for spammers to include fake ones. We should do so in the controller instead.

To test, submit an abuse report and make sure it still includes your IP address. Also check that you get redirected to the homepage.

Bonus: While removing the hidden field, I discovered a JavaScript error on the page: [Error] Error: LiveValidation::initialize - No element with reference or id of 'feedback_summary' exists!‌That's because the field is actually abuse_report_summary.

I fixed that, so now please test that if you click inside the “Brief summary of Terms of Service violation (required)” field and then click outside of it without entering text, you’ll get the LiveValidation error that says, “Please enter a brief summary of your message.”

Status

Assignee

Sarken

Reporter

Sarken

Roadmap

Misc

Priority

High

Affects versions

0.9.221

Fix versions

Components

BackEnd

Difficulty

Easy

Required Access Level

Policy & Abuse

Milestone

Internal 0.9