search

KyleAMathews / docker-nginx

Nginx Docker image for easy hosting of static sites
MIT License
336 stars 108 forks source link

Update to latest h5bp nginx settings #9

Closed sp90 closed 8 years ago

sp90 commented 8 years ago

Maybe an update to match the latest update by h5bp

KyleAMathews commented 8 years ago

Sounds great! Would you like to put together a PR?

KyleAMathews commented 8 years ago

It'd be good to include actual expire rules https://github.com/h5bp/server-configs-nginx/blob/66d0c463e0a347563692e3df735512b3babdf959/h5bp/location/expires.conf

sp90 commented 8 years ago

I can fix that up, give me some time i will get it together

KyleAMathews commented 8 years ago

💯

sp90 commented 8 years ago

@KyleAMathews wanna put them in root or?

KyleAMathews commented 8 years ago

I think keeping it in a separate file and then just including it makes sense. Neater + easier to browse/maintain.

sp90 commented 8 years ago
screen shot 2016-05-11 at 18 21 52

This is the structure i went for for now

KyleAMathews commented 8 years ago

👍