You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Patrick Hunt (JIRA)" <ji...@apache.org> on 2010/02/22 19:20:28 UTC

[jira] Created: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

c client doesn't allow ipv6 numeric connect string
--------------------------------------------------

                 Key: ZOOKEEPER-677
                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
             Project: Zookeeper
          Issue Type: Bug
          Components: c client
    Affects Versions: 3.2.2
            Reporter: Patrick Hunt
            Priority: Critical
             Fix For: 3.3.0


The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Open  (was: Patch Available)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12843616#action_12843616 ] 

Benjamin Reed commented on ZOOKEEPER-677:
-----------------------------------------

Committed revision 921439.


> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Patch Available  (was: Open)

retrying hudson.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Attachment: ZOOKEEPER-677.patch

the earlier patch doesnt compile on hudson machine and needs specific include for netdb,h header file.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12843445#action_12843445 ] 

Hadoop QA commented on ZOOKEEPER-677:
-------------------------------------

+1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12438242/ZOOKEEPER-677.patch
  against trunk revision 921201.

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs.  The patch does not introduce any new Findbugs warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    +1 core tests.  The patch passed core unit tests.

    +1 contrib tests.  The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h1.grid.sp2.yahoo.net/8/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h1.grid.sp2.yahoo.net/8/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h1.grid.sp2.yahoo.net/8/console

This message is automatically generated.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Attachment:     (was: ZOOKEEPER-677.patch)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Patrick Hunt (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated ZOOKEEPER-677:
-----------------------------------

    Status: Patch Available  (was: Open)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Reed updated ZOOKEEPER-677:
------------------------------------

    Status: Open  (was: Patch Available)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12837110#action_12837110 ] 

Hadoop QA commented on ZOOKEEPER-677:
-------------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12436677/ZOOKEEPER-677.patch
  against trunk revision 912119.

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs.  The patch does not introduce any new Findbugs warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    -1 core tests.  The patch failed core unit tests.

    +1 contrib tests.  The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/122/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/122/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/122/console

This message is automatically generated.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Attachment: ZOOKEEPER-677.patch

the issue is problem with getaddrinfo implementaiton on the hudson machine. According to the spec getaddrinfo should work on a ipv6 host with ai_flags=AI_ADDRCONFIG and ai_family=AF_UNSPEC, but looks like it returns EAI_ADDRFAMILY. 

I put in a retry with ai_flags=0 in case of the above errors. It seems like its a good optimization to use AI_ADDRCONFIG for AF_UNSPEC

http://www.ops.ietf.org/lists/v6ops/v6ops.2003/msg01377.html, so a retry should make it work in case of such faulty implementations of getaddrinfo.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Patrick Hunt (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated ZOOKEEPER-677:
-----------------------------------

    Status: Open  (was: Patch Available)

cancelling patch, hudson fails with:

     [exec]      [exec] Zookeeper_simpleSystem::testIPV6 : assertion : elapsed 1000

....

     [exec]      [exec] /grid/0/hudson/hudson-slave/workspace/Zookeeper-Patch-h8.grid.sp2.yahoo.net/trunk/src/c/tests/TestClient.cc:582: Assertion: assertion failed [Expression: zk]
     [exec]      [exec] Failures !!!
     [exec]      [exec] Run: 44   Failure total: 1   Failures: 1   Errors: 0
     [exec]      [exec] make: *** [run-check] Error 1

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar reassigned ZOOKEEPER-677:
---------------------------------------

    Assignee: Benjamin Reed

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Benjamin Reed
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Attachment: ZOOKEEPER-677.patch

this patch should fix the issue.


> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Assigned: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar reassigned ZOOKEEPER-677:
---------------------------------------

    Assignee: Mahadev konar  (was: Benjamin Reed)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12842194#action_12842194 ] 

Hadoop QA commented on ZOOKEEPER-677:
-------------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12438065/ZOOKEEPER-677.patch
  against trunk revision 919640.

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs.  The patch does not introduce any new Findbugs warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    -1 core tests.  The patch failed core unit tests.

    +1 contrib tests.  The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/134/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/134/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h8.grid.sp2.yahoo.net/134/console

This message is automatically generated.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Open  (was: Patch Available)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Open  (was: Patch Available)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Patch Available  (was: Open)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Reed updated ZOOKEEPER-677:
------------------------------------

    Hadoop Flags: [Reviewed]

+1 looks good

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Open  (was: Patch Available)

cancelling the patch... looks like getaddrinfo() does not set the errno value. So, we will have to set it manually in case of error conditions from getaddrinfo()

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Benjamin Reed
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Reed updated ZOOKEEPER-677:
------------------------------------

    Resolution: Fixed
        Status: Resolved  (was: Patch Available)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Patch Available  (was: Open)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Patrick Hunt (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated ZOOKEEPER-677:
-----------------------------------

    Attachment: ZOOKEEPER-677.patch

Fixed a problem on my machine (ubuntu karmic) - missing EAI_NODATA check.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12842739#action_12842739 ] 

Benjamin Reed commented on ZOOKEEPER-677:
-----------------------------------------

looks good mahadev. thanx for fixing. one problem:

{quote}
             LOG_ERROR(("getaddrinfo: %s\n", strerror(errno)));
+            errno = getaddrinfo_errno(rc);
{quote}

should be

{quote}
+            errno = getaddrinfo_errno(rc);
             LOG_ERROR(("getaddrinfo: %s\n", strerror(errno)));
{quote}

any idea why the test is failing?

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Reed updated ZOOKEEPER-677:
------------------------------------

    Status: Patch Available  (was: Open)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Hudson (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12844056#action_12844056 ] 

Hudson commented on ZOOKEEPER-677:
----------------------------------

Integrated in ZooKeeper-trunk #732 (See [http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/732/])
    . c client doesn't allow ipv6 numeric connect string


> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Hadoop QA (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12842862#action_12842862 ] 

Hadoop QA commented on ZOOKEEPER-677:
-------------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12438240/ZOOKEEPER-677.patch
  against trunk revision 919706.

    +1 @author.  The patch does not contain any @author tags.

    +1 tests included.  The patch appears to include 6 new or modified tests.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 findbugs.  The patch does not introduce any new Findbugs warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    -1 core tests.  The patch failed core unit tests.

    +1 contrib tests.  The patch passed contrib unit tests.

Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h1.grid.sp2.yahoo.net/2/testReport/
Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h1.grid.sp2.yahoo.net/2/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h1.grid.sp2.yahoo.net/2/console

This message is automatically generated.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Reed updated ZOOKEEPER-677:
------------------------------------

    Status: Patch Available  (was: Open)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Patrick Hunt (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated ZOOKEEPER-677:
-----------------------------------

    Status: Open  (was: Patch Available)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Patch Available  (was: Open)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Attachment: ZOOKEEPER-677.patch

this patch adds a gcc parameter so that all the return codes of getaddrinfo are defined. the tests pass on the hudson machine now.

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Mahadev konar (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mahadev konar updated ZOOKEEPER-677:
------------------------------------

    Status: Patch Available  (was: Open)

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch, ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (ZOOKEEPER-677) c client doesn't allow ipv6 numeric connect string

Posted by "Benjamin Reed (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ZOOKEEPER-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Benjamin Reed updated ZOOKEEPER-677:
------------------------------------

    Attachment: ZOOKEEPER-677.patch

this patch finds the port using strrchr instead of strchr to allow ipv6 addresses. the test case assumes that localhost is reachable with ::1

> c client doesn't allow ipv6 numeric connect string
> --------------------------------------------------
>
>                 Key: ZOOKEEPER-677
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-677
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: c client
>    Affects Versions: 3.2.2
>            Reporter: Patrick Hunt
>            Priority: Critical
>             Fix For: 3.3.0
>
>         Attachments: ZOOKEEPER-677.patch
>
>
> The c client doesn't handle ipv6 numeric addresses as they are colon : delmited. After splitting the host/port on : we look for the port as the second entry in the array rather than the last entry in the array.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.