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

Updated nginx configuration for kibana running with context path #26

Conversation

SachinKSingh28
Copy link
Contributor

This change is to be able to fix the issue described in #25

Also simplifies the rules that we have in tools-context for kibana.

This is dependent on https://github.com/SachinKSingh28/adop-kibana repository and image produced from it.

@@ -95,6 +72,5 @@ server {
proxy_redirect off;
proxy_pass http://ldap-phpadmin;
}
include /etc/nginx/sites-enabled/service-extension/*.conf;
Copy link
Contributor

Choose a reason for hiding this comment

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

Why we want to remove this line? It was introduced in #22

@SachinKSingh28 SachinKSingh28 force-pushed the feature/kibana-rule-fix branch from 3d3f781 to 74d7202 Compare November 9, 2016 20:53
@SachinKSingh28
Copy link
Contributor Author

@anton-kasperovich : Thanks for the review! I have added the server-extension configuration back in. It was accidentally removed. Also squashed the commits.

@SachinKSingh28
Copy link
Contributor Author

Closing this PR as the same issue has been fixed in PR #29

@svscorp
Copy link
Contributor

svscorp commented Sep 27, 2017

Yay!

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.

3 participants