-
Notifications
You must be signed in to change notification settings - Fork 161
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 config #47
Comments
I agree it is better practice to separate the Nginx site configuration to prevent it from being overwritten during BBB upgrading. This is also mentioned in the docs, however it requires for a separate HTTPS certificate. |
if you open /etc/nginx/sites-available/bigbluebutton that's why I mentioned it in previous message |
Hmm I never noticed that. That's an interesting idea! One thing personally though is that I sort of expect all Nginx configs to be in |
sure. no problem. |
That's good news. I think I'll fix the documentation to reflect your recommendation. Thanks! You are also welcome to submit a PR :) |
|
PR = Pull request |
I made it :) |
Great, thanks! Merged in #59 |
The better way to place nginx config for exporter is
/etc/bigbluebutton/nginx/metrics.nginx
it will not be overwritten during update
The text was updated successfully, but these errors were encountered: