[ 
https://issues.apache.org/jira/browse/CURATOR-583?focusedWorklogId=486628&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-486628
 ]

ASF GitHub Bot logged work on CURATOR-583:
------------------------------------------

                Author: ASF GitHub Bot
            Created on: 20/Sep/20 08:21
            Start Date: 20/Sep/20 08:21
    Worklog Time Spent: 10m 
      Work Description: hjyun328 commented on pull request #374:
URL: https://github.com/apache/curator/pull/374#issuecomment-695759670


    @eolivelli, @TisonKun
   Thanks for your review.
   I just added a test case for `withNewMembers`.


----------------------------------------------------------------
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:
[email protected]


Issue Time Tracking
-------------------

            Worklog Id:     (was: 486628)
    Remaining Estimate: 0h
            Time Spent: 20m  (was: 10m)

> Fix ArrayIndexOutOfBoundsException when passing empty list parameter to 
> reconfigure API
> ---------------------------------------------------------------------------------------
>
>                 Key: CURATOR-583
>                 URL: https://issues.apache.org/jira/browse/CURATOR-583
>             Project: Apache Curator
>          Issue Type: Bug
>          Components: Framework
>    Affects Versions: 5.1.0
>            Reporter: hyeong jun yun
>            Priority: Major
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Whenever I add zookeeper servers by using reconfig API of the curator client, 
> it always throw ArrayIndexOutOfBoundsException.
> If there are no servers to add or remove when using reconfigure API of 
> ZooKeeperAdmin, I think it must pass not empty list parameter but null to 
> reconfigure API. Because ZooKeeperAdmin tries to join strings by accessing 
> first index of the server list.
> Please refer to the link below.
> https://github.com/apache/zookeeper/blob/release-3.6.2/zookeeper-server/src/main/java/org/apache/zookeeper/admin/ZooKeeperAdmin.java#L267-L269
> https://github.com/apache/zookeeper/blob/release-3.6.2/zookeeper-server/src/main/java/org/apache/zookeeper/common/StringUtils.java#L57



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to