[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Resolution: Fixed Status: Resolved (was: Patch Available) Committed revision 739446. (closing on behalf of mahadev who committed this) > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.0, 3.0.1 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-16: --- Hadoop Flags: [Reviewed] > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.0, 3.0.1 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-16: --- Status: Patch Available (was: Open) > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.1, 3.0.0 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mahadev konar updated ZOOKEEPER-16: --- Attachment: ZOOKEEPER-16.patch this patch adds the check at the server only on create. also added a test ot verify it, > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.0, 3.0.1 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Attachment: ZOOKEEPER-16.patch Included Ben's suggestion regarding c client check. Also fixed a test broken by other commit. > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.0, 3.0.1 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Benjamin Reed updated ZOOKEEPER-16: --- Status: Open (was: Patch Available) > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.1, 3.0.0 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Status: Patch Available (was: Open) > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.1, 3.0.0 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Attachment: ZOOKEEPER-16.patch Remerged against the mainline. Not declaring throws IAE because it's a runtime exception - but I did document in the javadoc for the effected methods. > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.0, 3.0.1 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch, > ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Flavio Paiva Junqueira updated ZOOKEEPER-16: Status: Open (was: Patch Available) Patch is stale. > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.1, 3.0.0 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Affects Version/s: 3.0.0 3.0.1 Status: Patch Available (was: Open) > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.1, 3.0.0 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Attachment: ZOOKEEPER-16.patch Updated patch based on comment/feedback. We now throw IllegalArgumentException in the java client if the path is not valid In the c client we return a rc of ZBADARGUMENT (c client was already checking for null - just added the addl conditions) Tests added to both c and java. Note: valid path definition already existed in the programmer guide, no additional doc changes were necessary -- although I did update the java and c client API docs as appropriate. > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Affects Versions: 3.0.0, 3.0.1 >Reporter: Patrick Hunt >Assignee: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch, ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Attachment: ZOOKEEPER-16.patch this patch starts to validate the path, but quickly runs into an issue. create(//sync) handles validation fine, we throw KeeperException.BadArgumentException however create(//async) is not declared to throw KeeperException we can call the async callback handler with rc code of BadArgument, however this is suboptimal, it would be much better to throw an exception right away. We lose the error message as well (contained by the exception) our best option afiak is to deprecate the current async methods and create new async methods which throw KE and have "Path path" arguments instead of "String path" arguments. We could validate the path when Path is created (a new zk type), also passing around Path objects might allow us to optimize serialization/deser during the marshalling stages. It really is unfortunate that the async methods aren't declared to throw KeeperException, adding this would allow error handling flexibility in future (perhaps some other error type we haven't throught of, separate from path validation). Thoughts? > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Reporter: Patrick Hunt > Fix For: 3.1.0 > > Attachments: ZOOKEEPER-16.patch > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.
[jira] Updated: (ZOOKEEPER-16) Need to do path validation
[ https://issues.apache.org/jira/browse/ZOOKEEPER-16?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Patrick Hunt updated ZOOKEEPER-16: -- Fix Version/s: 3.1.0 Assignee: (was: Benjamin Reed) > Need to do path validation > -- > > Key: ZOOKEEPER-16 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-16 > Project: Zookeeper > Issue Type: Bug > Components: c client, java client, server >Reporter: Patrick Hunt > Fix For: 3.1.0 > > > Moved from SourceForge to Apache. > http://sourceforge.net/tracker/index.php?func=detail&aid=1963141&group_id=209147&atid=1008544 -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.