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

[nginx] Fix nginx warnings #4090

Closed
2 tasks done
acozine opened this issue Jul 28, 2023 · 4 comments
Closed
2 tasks done

[nginx] Fix nginx warnings #4090

acozine opened this issue Jul 28, 2023 · 4 comments
Assignees
Labels
newdev Operations pulls issues into the Operations ZenHub board

Comments

@acozine
Copy link
Contributor

acozine commented Jul 28, 2023

The load balancer currently warns us:

pulsys@lib-adc1:/etc/nginx$ sudo nginx -s reload
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/abid_prod.conf:23
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/cdh_prod_geniza.conf:46
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/cicognara-org.conf:25
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/iiif-staging.conf:23
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/imagecat_prod.conf:23
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/issuetracker.conf:24
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/lae-prod.conf:24
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/lib-mudd-prod.conf:59
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/lib-mudd-staging.conf:24
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/libimages.conf:33
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/library-prod.conf:75
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/libweb5.conf:12
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/locator-prod.conf:24
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/lrp.conf:23
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/maps-prod.conf:24
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/researchdata_prod.conf:33
nginx: [warn] protocol options redefined for 0.0.0.0:443 in /etc/nginx/conf.d/researchdata_staging.conf:24

This fix is to change listen 443 ssl; to listen 443 ssl http2; in the site config files.

Keep an inventory of fixed domains

  • catalog-prod.conf
  • catalog-staging.conf
@acozine acozine added the newdev label Jul 28, 2023
@amitduaaa
Copy link

I can work on this. Please assign this to me.

@acozine
Copy link
Contributor Author

acozine commented Jul 31, 2023

@iconichal thanks for offering to work on this issue! We can't assign an issue outside of our team, but if you open a PR we would at least consider merging it.

@acozine acozine changed the title Fix nginx warnings [nginx] Fix nginx warnings Aug 8, 2023
@leefaisonr
Copy link
Contributor

The original fix when running sudo nginx -s reload generates more errors.
We changed the nginxplus role in abid on line 24 from:

listen 443 ssl http2;

To look like:

   listen 443 ssl;

    http2 on;

We then tested this fix by running the playbook, and checking the site after to see if it was still up.

@acozine acozine added the Operations pulls issues into the Operations ZenHub board label Sep 11, 2023
kayiwa added a commit that referenced this issue Sep 20, 2023
christinach pushed a commit that referenced this issue Sep 20, 2023
* migrate to new postgresql server

* comment out bionic vms

* upgrade all vms to jammy

* fix nginx warnings

related to #4090
kayiwa added a commit that referenced this issue Oct 3, 2023
* migrate to new postgresql server

* comment out bionic vms

* upgrade all vms to jammy

* fix nginx warnings

related to #4090
@kayiwa
Copy link
Member

kayiwa commented Jan 22, 2025

closed by this work #5765

@kayiwa kayiwa closed this as completed Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
newdev Operations pulls issues into the Operations ZenHub board
Projects
None yet
Development

No branches or pull requests

4 participants