search

cdnjs / new-website

🖥 cdnjs.com website
https://cdnjs.com
ISC License
451 stars 212 forks source link

Remove HPKP headers #348

Closed klausenbusk closed 4 years ago

klausenbusk commented 4 years ago

HPKP isn't supported by any mainstream browsers these days[1][2], and the pins seems to be out-of-date.

[1] https://www.chromestatus.com/feature/5903385005916160 [2] https://bugzilla.mozilla.org/show_bug.cgi?id=1412438

MattIPv4 commented 4 years ago

👋 API server here is legacy and not in production, it has been replaced with https://github.com/cdnjs/api-server/blob/master/app.js -- would you be happy to mirror this change over there?

The web server here is still in production but I'd also consider it be a legacy app, the WIP replacement at https://github.com/cdnjs/static-website/blob/master/server/app.js is a barebones express server that has no pinning :)

klausenbusk commented 4 years ago

wave API server here is legacy and not in production, it has been replaced with https://github.com/cdnjs/api-server/blob/master/app.js -- would you be happy to mirror this change over there?

Sure: https://github.com/cdnjs/api-server/pull/25