search

DarkShield / daDashboard

Dashboard Frontend
0 stars 0 forks source link

Implement Helmet #87

Closed mattjay closed 9 years ago

mattjay commented 9 years ago

Helmet required in server.js Added to list of dependencies in package.json Added X-Frame-Options:SAMEORIGIN Added X-XSS-Protection:1; mode=block Suppressed the X-Powered-By: Express header

mattjay commented 9 years ago

@ProZachJ - Sanity check me that its ok that I stuck all these app.use(helmet.whatever()) on the top of the middleware list. I tried it on the bottom but they didn't work.

mattjay commented 9 years ago

Fixes #79

ProZachJ commented 9 years ago

@mattjay looks like I've got this passing. Do you want to go ahead and merge it or did you have some additional tasks you wanted to complete first?

ProZachJ commented 9 years ago

this is now deployed on http://strider.vicetek.com:8888/

mattjay commented 9 years ago

closing this duplicate ticket. merged to master in #89