Page MenuHomeDevCentral

Honour nginx rewrite directive
ClosedPublic

Authored by dereckson on Feb 16 2018, 22:57.
Tags
None
Referenced Files
F46009789: D1315.diff
Mon, Aug 24, 15:02
F45980254: D1315.id3376.diff
Mon, Aug 24, 11:23
Unknown Object (File)
Sat, Aug 22, 10:49
Unknown Object (File)
Mon, Aug 17, 23:00
Unknown Object (File)
Mon, Aug 17, 14:06
Unknown Object (File)
Mon, Aug 17, 08:26
Unknown Object (File)
Sun, Aug 16, 21:56
Unknown Object (File)
Sun, Aug 16, 12:12
Subscribers
None

Details

Summary

To use DOCUMENT_URI instead of REQUEST_URI allow
this rewrite rule in nginx, introduced in D1314:

rewrite ^/servers\-log/?(.*)$ /$1 break;

This allows to serve the service at /servers-log,
but to read the document URI relative to the service root.

An immediate benefit is to separate concerns:

  • the web server is responsible to define the mounting point and to give to the service a clean URL,
  • the service is responsible to handle a request, taking in consideration this clean URL.

This change is needed by ad35fd26f9a9.

Test Plan

Tested with ad35fd26f9a9 configuration.

Diff Detail

Repository
rAPISRVLOGS Servers log :: Add new entries API
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
honor-nginx-rewrite (branched from master)
Build Status
Buildable 2080
Build 2328: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.Feb 16 2018, 22:58
This revision was automatically updated to reflect the committed changes.