Page MenuHomePhabricator

Add a linter rule for the use of `parse_str`
ClosedPublic

Authored by joshuaspence on Aug 11 2015, 2:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 3:15 PM
Unknown Object (File)
Fri, Dec 13, 8:19 AM
Unknown Object (File)
Wed, Dec 4, 1:05 PM
Unknown Object (File)
Wed, Nov 27, 7:46 AM
Unknown Object (File)
Nov 23 2024, 11:23 AM
Unknown Object (File)
Nov 19 2024, 11:46 AM
Unknown Object (File)
Nov 15 2024, 8:11 AM
Unknown Object (File)
Nov 11 2024, 3:34 PM
Subscribers

Details

Summary

The use of the parse_str method (when called without sepcifying a second parameter) hinders static analysis. Specifically, the parse_str('...') behaves similarly to extract.

Test Plan

Added unit tests.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence retitled this revision from to Add a linter rule for the use of `parse_str`.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Aug 11 2015, 2:56 AM
This revision was automatically updated to reflect the committed changes.