Page MenuHomePhabricator

Update the celerity resource controller to be module-aware
AbandonedPublic

Authored by hach-que on Dec 11 2013, 3:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 4, 5:44 AM
Unknown Object (File)
Mar 27 2024, 4:41 PM
Unknown Object (File)
Mar 24 2024, 10:25 PM
Unknown Object (File)
Mar 24 2024, 10:16 PM
Unknown Object (File)
Mar 18 2024, 3:44 AM
Unknown Object (File)
Jan 15 2024, 11:54 PM
Unknown Object (File)
Jan 15 2024, 9:12 AM
Unknown Object (File)
Jan 15 2024, 9:11 AM

Details

Summary

Depends on D7753. This updates the celerity resource controller so that it will serve up resources located in external libphutil modules.

Test Plan

Ensured that Phabricator resources still rendered correctly (I did break it at one point so I know they're working again). Also went to a URL in another module's resource map and Phabricator served it up correctly.

Diff Detail

Branch
celerity-external-lookup
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

src/infrastructure/celerity/CelerityResourceController.php
63–67

I don't know what performance impacts this change might have...

Throwing this out since @epriestley is implementing a better version.