should return error when NewURL failed.

Signed-off-by: imxyb <xyb4638@gmail.com>
apache · Nov 26, 2019 · e4561b6 · e4561b6
1 parent 6a691cd
commit e4561b6
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/config_center/parser/configuration_parser.go b/config_center/parser/configuration_parser.go
@@ -134,14 +134,14 @@ func serviceItemToUrls(item ConfigItem, config ConfiguratorConfig) ([]*common.UR
 				newUrlStr = newUrlStr + v
 				url, err := common.NewURL(context.Background(), newUrlStr)
 				if err != nil {
-					perrors.WithStack(err)
+					return nil, perrors.WithStack(err)
 				}
 				urls = append(urls, &url)
 			}
 		} else {
 			url, err := common.NewURL(context.Background(), urlStr)
 			if err != nil {
-				perrors.WithStack(err)
+				return nil, perrors.WithStack(err)
 			}
 			urls = append(urls, &url)
 		}