[wp-trac] [WordPress Trac] #37192: Validate before sanitizing when processing REST Request arguments

WordPress Trac noreply at wordpress.org
Thu Jun 30 22:03:31 UTC 2016


#37192: Validate before sanitizing when processing REST Request arguments
--------------------------------------+-----------------------------
 Reporter:  danielbachhuber           |       Owner:
     Type:  defect (bug)              |      Status:  new
 Priority:  normal                    |   Milestone:  Future Release
Component:  REST API                  |     Version:  4.4
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+-----------------------------
Changes (by schlessera):

 * keywords:  needs-unit-tests needs-patch => has-patch has-unit-tests


Comment:

 I have written a test that failed with the currently committed behavior
 (first sanitizing, then validating), and then made the fix to first
 validate, and then sanitize, which makes the test succeed again.

 This is my first patch submission, so please let me know if I missed
 something.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/37192#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list