germanattanasio / visual-recognition-nodejs

DEPRECATED: this repo is no longer actively maintained
Apache License 2.0
361 stars 351 forks source link

Modify CSP to allow google analytics to function #234

Closed kognate closed 8 years ago

kognate commented 8 years ago

I had to add unsafe-inline or else do major surgery to how we hare handling the env vars for GA.

nfriedly commented 8 years ago

We probably shouldn't enable unsafe-inline just for google analytics. I'd rather put it in a standalone file and then hard-code the ID. (Or, if you want to get fancy, put it into it's own jade template and create a new endpoint to run it from.)