Page MenuHomePhabricator

Remove Composer support from XHProf
ClosedPublic

Authored by epriestley on Aug 27 2014, 5:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 9:44 AM
Unknown Object (File)
Thu, Apr 25, 3:21 AM
Unknown Object (File)
Sat, Apr 6, 6:48 AM
Unknown Object (File)
Fri, Apr 5, 11:27 PM
Unknown Object (File)
Mar 24 2024, 5:23 AM
Unknown Object (File)
Mar 20 2024, 2:54 PM
Unknown Object (File)
Feb 3 2024, 9:43 PM
Unknown Object (File)
Jan 30 2024, 8:29 PM
Subscribers
Tokens
"The World Burns" token, awarded by btrahan.

Details

Summary

See https://github.com/phacility/xhprof/pull/40. This is currently broken and extremely insecure.

We may want to restore it eventually, but understanding composer is very complex (no one who touched this realized that the package was owned by someone unrelated to the project who can apparently redirect it at will with no accountability). No one on the ticket seems to have any reason why this isn't totally wide open, and I haven't gotten in touch with anyone in #composer-dev.

Test Plan

N/A

Diff Detail

Repository
rXPRF XHProf
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Remove Composer support from XHProf.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan awarded a token.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 27 2014, 5:15 PM
epriestley updated this revision to Diff 24953.

Closed by commit rXPRFe3df210a649b (authored by @epriestley).