Merge pull request #6 from karimra/5-path-flag

removed defatult "/" path and marked path mandatory
karimra · Apr 23, 2020 · 0b6bb48 · 0b6bb48
2 parents 9e80070 + c49d6b1
commit 0b6bb48
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/cmd/get.go b/cmd/get.go
@@ -226,7 +226,8 @@ var getCmd = &cobra.Command{
 func init() {
 	rootCmd.AddCommand(getCmd)
 
-	getCmd.Flags().StringSliceP("path", "", []string{"/"}, "get request paths")
+	getCmd.Flags().StringSliceP("path", "", []string{""}, "get request paths")
+	getCmd.MarkFlagRequired("path")
 	getCmd.Flags().StringP("prefix", "", "", "get request prefix")
 	getCmd.Flags().StringP("model", "", "", "get request model")
 	getCmd.Flags().StringP("type", "t", "ALL", "the type of data that is requested from the target. one of: ALL, CONFIG, STATE, OPERATIONAL")

diff --git a/cmd/subscribe.go b/cmd/subscribe.go
@@ -193,7 +193,8 @@ var subscribeCmd = &cobra.Command{
 func init() {
 	rootCmd.AddCommand(subscribeCmd)
 	subscribeCmd.Flags().StringP("prefix", "", "", "subscribe request prefix")
-	subscribeCmd.Flags().StringSliceP("path", "", []string{"/"}, "subscribe request paths")
+	subscribeCmd.Flags().StringSliceP("path", "", []string{""}, "subscribe request paths")
+	subscribeCmd.MarkFlagRequired("path")
 	subscribeCmd.Flags().Int32P("qos", "q", 20, "qos marking")
 	subscribeCmd.Flags().BoolP("updates-only", "", false, "only updates to current state should be sent")
 	subscribeCmd.Flags().StringP("subscription-mode", "", "stream", "one of: once, stream, poll")