-
Notifications
You must be signed in to change notification settings - Fork 58
[HttpPlatformHandler]: Forward common headers from reverse proxy #2
Comments
Also of interest are the remote ip & port and the reverse proxy public IP & port. I'd recommend doing it all in middleware. Eventually we'll need the TLS token binding. |
From @davidfowl on September 11, 2015 16:41 I think scheme is the only one we'd want to support by default. |
HttpPlatformHandler needs to forward them all, it's up to the app how many forwarders they check. |
From @davidfowl on September 11, 2015 17:13 We'll let the user experience drive that. This is our new ootb experience for IIS and it can't be degraded in obvious ways. We'll discuss it in the design meeting |
A few other interop questions: |
From @davidfowl on September 15, 2015 3:43 We decided to make this a separate middleware This middleware also needs to handle windows auth via the |
From @rubenprins on September 16, 2015 17:39 What about nested applications/child paths? Is that still something that will be supported out of the box? Or will we need to configure the application's base path (a.k.a. |
From @davidfowl on September 16, 2015 17:51 Won't be supported out of the box. I haven't tried it myself with the platform handler. |
@davidfowl repo name? |
From @davidfowl on September 27, 2015 5:7 @Tratcher move this bug over |
# This is the 1st commit message: fix app recycle logic # The commit message #2 will be skipped: # adding parallel shutdown logic # The commit message #3 will be skipped: # some update # The commit message #4 will be skipped: # more change # The commit message #5 will be skipped: # close hkey # The commit message #6 will be skipped: # update logging # The commit message #7 will be skipped: # remove unused code # The commit message #8 will be skipped: # format change # The commit message #9 will be skipped: # exclude not-subapp case while sudstring does match # Please enter the commit message for your changes. Lines starting
# This is the 1st commit message: Add gtest! # The commit message #2 will be skipped: # still plumming gtest stuff, TODO consider just cloning the gTestRepo.
From @davidfowl on September 11, 2015 16:11
This bug is to track the work that needs to happen so that items forwarded from httpplatformhandler show up in the user's application.
There are several parts to this:
We could do some of it in middleware (like WindowsAuth) but that would mean it wouldn't work out of the box. It could also be in the template by default so that we don't couple to the HttpPlatformHandler in any of our core components.
Copied from original issue: aspnet/Hosting#349
The text was updated successfully, but these errors were encountered: