Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace HttpListener for .NET 6.0+ #142

Open
wants to merge 2 commits into
base: version-six
Choose a base branch
from

Conversation

pgrawehr
Copy link

This replaces the legacy System.Net.HttpListener with SpaceWizzards.HttpListener.HttpListener for .NET 6.0 and above. This replacement implementation is fully managed and does not use any poorly maintained Windows IIS components under the hood. It also no longer requires admin rights (or complex netsh rules) to create the listener and "just works". The only probable downside I can think of is that the network port cannot be shared directly with this implementation, but when something as lightweight as Grapevine is used, that's probably not intended anyway.

Also includes some fixes for ContentType, the old implementation for the default list wasn't working. This resulted in html files being delivered as binary, which then were downloaded by the browser as files instead of displayed as content.

@scottoffen
Copy link
Owner

I like this idea, but I'm unfamiliar with the package. I'll look into this and get back.

@pgrawehr
Copy link
Author

I like this idea, but I'm unfamiliar with the package. I'll look into this and get back.

I didn't know it, either. But it appears to be working and (despite it's low version number) has quite a large number of downloads already.

@Hyrius
Copy link

Hyrius commented Sep 3, 2023

Space Wizards are the developers of the Space Station 14 game.
I use a fork of another project maintained by them.

@Hyrius
Copy link

Hyrius commented Oct 26, 2023

I couldn't get HTTPS to work with SpaceWizzards.HttpListener.HttpListener.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants