You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by GitBox <gi...@apache.org> on 2021/01/26 16:51:14 UTC
[GitHub] [camel] colin-mullikin opened a new pull request #4937: Fix Consul key path used when determining leader.
colin-mullikin opened a new pull request #4937:
URL: https://github.com/apache/camel/pull/4937
The session lock is on the full key path, not on the root path. As far as I can tell, the result of this method isn't actually used anywhere, but this fix at least makes it correct.
I discovered this issue due to my Consul ACL policy. My application has access to the full key path, but does not have access to the configured root path, so I was getting an error at startup.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] colin-mullikin commented on pull request #4937: Fix Consul key path used when determining leader.
Posted by GitBox <gi...@apache.org>.
colin-mullikin commented on pull request #4937:
URL: https://github.com/apache/camel/pull/4937#issuecomment-768393404
Last push was just to add the Apache header to my test file.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] davsclaus merged pull request #4937: Fix Consul key path used when determining leader.
Posted by GitBox <gi...@apache.org>.
davsclaus merged pull request #4937:
URL: https://github.com/apache/camel/pull/4937
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [camel] lburgazzoli commented on pull request #4937: Fix Consul key path used when determining leader.
Posted by GitBox <gi...@apache.org>.
lburgazzoli commented on pull request #4937:
URL: https://github.com/apache/camel/pull/4937#issuecomment-767685044
@colin-mullikin thx for spotting this, is there any chance you also provide a test ?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org