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

Fixes for Windows clients with Enterprise #544

Merged
merged 9 commits into from
Oct 26, 2016

Conversation

jacobmw
Copy link
Contributor

@jacobmw jacobmw commented Sep 1, 2016

No description provided.

@jaxxstorm
Copy link
Contributor

This looks good, can I ask why 85dd5d6 was added though? We've gone back and forwards on the default vhost setting. I believe it should have a preceding slash, unless there's a good justification, can you please revert that particular commit?

@jacobmw
Copy link
Contributor Author

jacobmw commented Oct 25, 2016

Ah, sorry, didn't mean to reset the default. I have put it back to have the slash.

On Oct 23, 2016, at 1:44 AM, Lee Briggs notifications@github.com wrote:

This looks good, can I ask why 85dd5d6 85dd5d6 was added though? We've gone back and forwards on the default vhost setting. I believe it should have a preceding slash, unless there's a good justification, can you please revert that particular commit?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #544 (comment), or mute the thread https://github.com/notifications/unsubscribe-auth/AL6_SL5sL3TYFhB_cxNmNYYv4O7o1gWzks5q2x6AgaJpZM4JyTVx.

@jaxxstorm
Copy link
Contributor

Awesome, thanks for the contribution!

@jaxxstorm jaxxstorm merged commit 30ffce5 into sensu:master Oct 26, 2016
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