Page MenuHomePhabricator

[server] Use baseRoutePath in getAppURLFactsFromRequestURL
ClosedPublic

Authored by ashoat on Mar 29 2022, 3:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 29, 2:47 PM
Unknown Object (File)
Mon, Jun 24, 4:10 AM
Unknown Object (File)
Sun, Jun 23, 6:11 PM
Unknown Object (File)
Fri, Jun 21, 11:58 PM
Unknown Object (File)
Fri, Jun 21, 11:58 PM
Unknown Object (File)
Fri, Jun 21, 11:57 PM
Unknown Object (File)
Fri, Jun 21, 11:40 PM
Unknown Object (File)
Thu, Jun 20, 10:19 PM

Details

Summary

baseRoutePath is the internal one, and req.url represents Node's internal path. basePath refers to the public (Apache) path.

Depends on D3552

Test Plan
  1. We tested the landing page, the SquadCal app, and the Comm app
  2. We made sure we tried configs where baseRoutePath !== basePath
  3. We also checked view-source to make sure things were set correctly
  4. We made sure there were no errors in the browser console
  5. We played around with the site, logging in and navigating and making sure things worked correctly
  6. We tried loading a specific URL for a specific chat directly

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable