Skip to content
This repository was archived by the owner on Nov 1, 2018. It is now read-only.

AspNetCoreModule doesn't work with WebListener/Http.Sys #8

Closed
Tratcher opened this issue Oct 5, 2015 · 4 comments
Closed

AspNetCoreModule doesn't work with WebListener/Http.Sys #8

Tratcher opened this issue Oct 5, 2015 · 4 comments
Labels
0 - Backlog AspNetCoreModule
Milestone
Backlog

Comments

@Tratcher
Copy link
Member

Tratcher commented Oct 5, 2015

Right now all the tests run with Kestrel. When I briefly tried WebListener some of the test cases failed. The logs showed WebListener starting fine, but it never served any requests and HttpPlatformHandler kept re-starting the back-end process until it gave up.

I've seen it work when published to full IIS, so it might be a test setup issue.
@Schlurcher
Copy link

Can confirm this is happening with 1.2 on full IIS 8 on Server 2012. Backend keeps restarting until IIS gives up and serves a 502 Bad Gateway.

@Tratcher Tratcher added the Task label Nov 18, 2015
@muratg muratg added this to the 1.0.0-rc2 milestone Nov 20, 2015
@pakrym pakrym removed the 2 - Working label Dec 1, 2015
@muratg
Copy link
Contributor

muratg commented Jan 21, 2016

Moving this to Backlog as we will be in RC2 ask mode very soon. If you feel strongly about this issue, please ping me.

@muratg muratg modified the milestones: Backlog, 1.0.0-rc2 Jan 21, 2016
@Tratcher
Copy link
Member Author

Ok, now we understand why it doesn't work, and it won't any time soon.

HttpPlatformHandler starts the back-end process (dnx.exe). Then it opens a socket and tries to connect to the specified port. When that succeeds it cross checks the process that opened the port with it's list of child processes. If it finds a mismatch then it assumes it has a port conflict and it ends all of it's child processes and starts over. In the case of WebListener the port is actually opened by Http.Sys in the System process, so it never matches and always gets ended by platform handler.

HttpPlatformHandler will eventually need to support Http.Sys based back-end servers, but it's a non-trivial change for them so this isn't happening any time soon.

@Tratcher Tratcher removed the Task label Jan 29, 2016
@Tratcher Tratcher changed the title Test with WebListener HttpPlatformHandler doesn't work with WebListener/Http.Sys Feb 1, 2016
This was referenced Feb 11, 2016
Closed
Closed
This was referenced Mar 8, 2016
Closed
Closed
@Tratcher Tratcher mentioned this issue Mar 28, 2016
Closed
@Tratcher Tratcher changed the title HttpPlatformHandler doesn't work with WebListener/Http.Sys AspNetCoreModule doesn't work with WebListener/Http.Sys Jun 14, 2016
@Tratcher Tratcher mentioned this issue Oct 27, 2016
Closed
@Tratcher
Copy link
Member Author

This issue was moved to aspnet/AspNetCoreModule#23

jkotalik pushed a commit that referenced this issue Mar 2, 2018
@jkotalik
# This is a combination of 9 commits.
7bc5983 
# 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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
0 - Backlog AspNetCoreModule
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
4 participants
@Schlurcher @pakrym @muratg @Tratcher