[PATCH] D93114: LNT: fix broken relative redirect

Tamar Christina via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 11 05:58:23 PST 2020


tnfchris created this revision.
tnfchris added a reviewer: thopre.
tnfchris requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

RFC 2616 has at some point an ambiguous statement in it that made it
look like relative URIs was not allowed in the Location header.

Some implementations like Flask implemented their response object
based on this mistake.  This ambiguity was later corrected in updated
RFC 2616 but Flask has kept the old implementation.   Which means to get
correct standards compliant redirects we need to override the default
responder and tell it not to force absolute URIs.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D93114

Files:
  lnt/server/ui/globals.py
  lnt/server/ui/regression_views.py
  lnt/server/ui/views.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93114.311202.patch
Type: text/x-patch
Size: 8706 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201211/b26d4671/attachment.bin>


More information about the llvm-commits mailing list