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

Server-side Updates #8

Merged
merged 3 commits into from
Feb 28, 2018
Merged

Server-side Updates #8

merged 3 commits into from
Feb 28, 2018

Conversation

seidior
Copy link
Collaborator

@seidior seidior commented Feb 28, 2018

We're switching HTTP servers from pushstate-server to Apache. Here's why:

Downsides of pushstate-server

Previously, we've been using pushstate-server as our HTTP server for serving up the production-ready site, as recommended by React. It's a simple HTTP server that works with a specific HTML5 feature called Pushstate.

However, this is not how pushstate-server was really designed. It's not designed to be run on port 80 as the primary production server for a website. I've had to employ a series of hacks to allow pushstate to serve up content on port 80 while not running it as root. I also don't know about the security implications of this.

Furthermore, pushstate-server doesn't allow for HTTPS connections. There are some alpha-stage projects that provide this and even a pull request, but they are young and untested.

Upsides of Apache

Switching to Apache means the site will behave more like any other site will. This means that assets will get served up in a standard way, the site assets can be combined with any other scripting language (like PHP), and Apache modules (rewrite, ssl, and others) will work with the portfolio. Guides for securing Apache installations are common, and Apache modules are very mature and reliable.

Ultimately, this means the project will be better-maintained and more robust. The web server should automatically restart when the server reboots. HTTPS will be supported. And we have flexibility for the server to behave... normally.

@seidior seidior merged commit 75c5a39 into gitmarkhubmunar:master Feb 28, 2018
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.

1 participant