GitHub user joshng opened a pull request:
https://github.com/apache/curator/pull/284
CURATOR-489: CreateBuilderImpl assigns protectedId if doProtected is â¦
This is a light change to correct a bug when `doProtected` is true: the
existing code sets `CreateBuilderImpl.doProtected = true`, but assigns
`protectedId = null`, resulting in a non-unique protection-token (`_c_null-`)
being applied to the name created in zookeeper. Given that all replicas
coordinating via zookeeper would use the identical token, this could
conceivably lead to incorrect behavior when the protection logic is needed.
While I believe this PR fixes the bug, I'd really suggest a more
comprehensive change to eliminate the `CreateBuilderImpl.doProtected` field
entirely, instead relying upon `protectedId != null` to convey this intention.
However, being unfamiliar with the nuances of the existing implementation, I
didn't want to perform that surgery today. If you think that change seems
preferable, I'd be happy to do that instead.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/joshng/curator CURATOR-489-set-protectedId
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/curator/pull/284.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #284
----
commit 5c27ecc29c3503633462cd95971ceb6a91175404
Author: josh gruenberg <joshgr@...>
Date: 2018-12-04T21:17:15Z
CURATOR-489: CreateBuilderImpl assigns protectedId if doProtected is true
----
---
