Merge pull request #15 from IvanKuchin/development

code cleanup and add few more checks
IvanKuchin · Mar 9, 2024 · fbd1422 · fbd1422
2 parents 45051cc + a8623f8
commit fbd1422
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -4,6 +4,7 @@
     // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
     "version": "0.2.0",
     "configurations": [
+
         {
             "name": "get cisco-dhcp",
             "type": "go",

diff --git a/internal/cli/get.go b/internal/cli/get.go
@@ -55,6 +55,10 @@ func getIPAddressesBelongsToSubnets(client api_client.Authenticator, cfg *config
 
 	addresses := IPAddresses{}
 
+	if len(cfg.Ipam_subnets) == 0 {
+		return addresses, fmt.Errorf("no subnets defined in config file")
+	}
+
 	for _, subnet := range cfg.Ipam_subnets {
 
 		subnet_id, err := getSubnetID(client, config_reader.Cfg, subnet)
@@ -196,6 +200,10 @@ func getCiscoDHCPOutputBySubnet(addresses IPAddresses, subnet string) (string, e
 func getCiscoDHCPOutput(client api_client.Authenticator, cfg *config_reader.Config) (string, error) {
 	output := ""
 
+	if len(cfg.Ipam_subnets) == 0 {
+		return "", fmt.Errorf("no subnets defined in config file")
+	}
+
 	for _, subnet := range cfg.Ipam_subnets {
 
 		subnet_id, err := getSubnetID(client, config_reader.Cfg, subnet)