[ZOOKEEPER-2757] Incorrect path crashes zkCli

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 3.5.3
• Fix Version/s: 3.5.4, 3.6.0
• Assignee: Abraham Fine
• Reporter: Flavio Paiva Junqueira
• Created: 14/Apr/17 21:58
• Updated: 26/May/17 23:44
• Resolved: 26/May/17 22:52

Description

If I try delete test without the leading /, then the CLI crashes with this exception:

Exception in thread "main" java.lang.IllegalArgumentException: Path must start with / character
	at org.apache.zookeeper.common.PathUtils.validatePath(PathUtils.java:51)
	at org.apache.zookeeper.ZooKeeper.delete(ZooKeeper.java:1659)
	at org.apache.zookeeper.cli.DeleteCommand.exec(DeleteCommand.java:83)
	at org.apache.zookeeper.ZooKeeperMain.processZKCmd(ZooKeeperMain.java:655)
	at org.apache.zookeeper.ZooKeeperMain.processCmd(ZooKeeperMain.java:586)
	at org.apache.zookeeper.ZooKeeperMain.executeLine(ZooKeeperMain.java:370)
	at org.apache.zookeeper.ZooKeeperMain.run(ZooKeeperMain.java:330)
	at org.apache.zookeeper.ZooKeeperMain.main(ZooKeeperMain.java:292)

It should really fail the operation rather than crash the CLI.

Issue Links

• links to: GitHub Pull Request #240