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

Demo web #381

Merged
merged 5 commits into from
Dec 17, 2024
Merged

Demo web #381

merged 5 commits into from
Dec 17, 2024

Conversation

pwalczysko
Copy link
Member

As discussed, this PR is:

  1. upgrading the ciphers and protocols in demo playbook
  2. upgrading nginx version
  3. removing unused confusing jinja files

cc @jburel

@pwalczysko
Copy link
Member Author

This has been now implemented on demo server.
All works fine (omero-web, nginx on 1.26.2 and the ciphers added).

cc @jburel

Happy for me to merge ?

@pwalczysko pwalczysko requested a review from jburel December 13, 2024 16:47
@pwalczysko
Copy link
Member Author

pwalczysko commented Dec 16, 2024

demo is now on

    nginx 1.26.2

has the

ssl_protocols   TLSv1.2 TLSv1.3;
    ssl_ciphers ECDHE-ECDSA-AES128-GCM-SHA256:ECDHE-RSA-AES128-GCM-SHA256:ECDHE-ECDSA-AES256-GCM-SHA384:ECDHE-RSA-AES256-GCM-SHA384:ECDHE-ECDSA-CHACHA20-POLY1305:ECDHE-RSA-CHACHA20-POLY1305:DHE-RSA-AES128-GCM-SHA256:DHE-RSA-AES256-GCM-SHA384:DHE-RSA-CHACHA20-POLY1305;
has the
omero.glacier2.IceSSL.Ciphers: "HIGH:!DHE"

line in the server config

@pwalczysko
Copy link
Member Author

demo has now all the web apps and the server and omero py updated

@pwalczysko
Copy link
Member Author

pwalczysko commented Dec 17, 2024

@jburel would you be happy for me to merge this please ? In order to minimize the complexity - all is deployed and working. (you have applied your manual changes on top of this). Thanks

After this (in the new year), we should discuss about how to get about upgrading OMERO.server whilst not interfering with the omero-certificate changes.

@jburel jburel merged commit c01729a into ome:master Dec 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants