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

HSTS in either nginx or Traefik? #270

Closed
g7morris opened this issue Feb 9, 2023 · 4 comments
Closed

HSTS in either nginx or Traefik? #270

g7morris opened this issue Feb 9, 2023 · 4 comments
Assignees
Labels

Comments

@g7morris
Copy link
Contributor

g7morris commented Feb 9, 2023

@nigelgbanks @jefferya Would either of both of you be game to try something here or Nigel perhaps include this with the upcoming release?


Have been testing sites on ssllabs and finding while we can get an "A rating" for the most part, it would be "nice to have" the possible A+

One way that might be helpful is to use HSTS headers in traefik?

As an example, when using nginx, one would add a header to ngnix's config or domain conf / profile.

server {
listen 443 ssl;
add_header Strict-Transport-Security "max-age=31536000; includeSubDomains" always;

The challenge I believe with nginx is that ISLE doesn't listen on port 443 or use ssl in the backend (right?) so perhaps instead we could review the following for traefik ?

Here are a couple of sources, I've dug up so far but am not totally clear on how to implement.

My main concern is the impact to the backend nginx / drupal service.

@nigelgbanks
Copy link
Contributor

Seems like we'll have to do that traefik side, since it is performing the SSL termination. So in the isle-dc repo would be the spot for it.

@jefferya
Copy link
Contributor

Agree with Nigel. Seems related to the TLS ticket in Isle-dc (Islandora-Devops/isle-dc#263). I've done some initial testing with min TLS 1.2 and HSTS enabled (Qualys SSL Labs A+ rating). Also, I'm using image tag: issue-215-update-alpine (the PHP 8.1 etc. unreleased images). The results are promising (though Fedora, code-server & Blazegraph are outside my current test scope). I'll be doing more testing over the next week.

@nigelgbanks
Copy link
Contributor

@jefferya you can switch to the main tag it has some more fixes since #238 has been merged.

@g7morris
Copy link
Contributor Author

Closing this as solved in isle-dc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants