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.

Sign up for GitHub

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

Jump to bottom

Problem with multiple Upstream servers #893

Merged
merged 1 commit into from
Oct 7, 2018
Merged

Problem with multiple Upstream servers #893

merged 1 commit into from
Oct 7, 2018

Conversation

juliocbc
Copy link
Contributor

@juliocbc juliocbc commented Oct 6, 2018

I've some trouble while trying with multiple upstream servers:

nginx: [emerg] "upstream" directive is not allowed here in /usr/local/etc/nginx/nginx.conf:56
nginx: configuration file /usr/local/etc/nginx/nginx.conf test failed

I've made a small change to make it work.

@juliocbc
Problem with multiple Upstream servers
82a47c3 
Small change to make it work.
@fabianfrz fabianfrz self-assigned this Oct 6, 2018
@fabianfrz fabianfrz added the bug Production bug label Oct 6, 2018
fabianfrz
fabianfrz approved these changes Oct 6, 2018
View reviewed changes
Copy link
Member

@fabianfrz fabianfrz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks.

@fichtner please include it in 1.1 stable

@fabianfrz fabianfrz merged commit d6c61bd into opnsense:ngx_rule_download Oct 7, 2018
fabianfrz added a commit that referenced this pull request Oct 7, 2018
@fabianfrz
www/nginx: import naxsi core rules into configuration (#891)
a2c119e 
* www/nginx: add draft of code to download naxsi rules
* www/nginx: run bmake style-fix
* www/nginx: naxsi rule download
* www/nginx: rename file
* www/nginx: add configd task
* www/nginx: add initial GUI code for rule download
* www/nginx: add view part
* www/nginx: fix inverted regex flag
* Problem with multiple Upstream servers (#893)

Small change to make it work.
@juliocbc juliocbc deleted the ngx_rule_download branch October 8, 2018 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fabianfrz fabianfrz fabianfrz approved these changes

@fichtner fichtner fichtner approved these changes

Assignees

@fabianfrz fabianfrz

Labels
bug Production bug
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@juliocbc @fichtner @fabianfrz