Service_api.go triggers aggregator error: Get "http:///resources/initpull": http: no Host in request URL

[sonyafenge]

Repro steps:

1. start service using the command below:

/usr/local/go/bin/go run resource-management/cmds/service-api/service-api.go  --master_ip=sonya-grs-server-us-central1-a-mig-mfl5 --resource_urls=34.168.53.159:9119,35.236.3.51:9119, -v=3  > ~/logs/sonya-grs-server-us-central1-a-mig-mfl5.log 2>&1

2. if resource_urls has one more "," at the end of string(--resource_urls=34.168.53.159:9119,35.236.3.51:9119,), service logs keep display this error, if using "--resource_urls=34.168.53.159:9119,35.236.3.51:9119", error will not display.

E0726 03:47:18.686107    2134 aggregator.go:164] Get "http:///resources/initpull": http: no Host in request URL

3. expected result: no error displayed

[yb01]

args need to be validated to trim any leading or trailing "," separators.

[yb01]

@sonyafenge , please verify.

[sonyafenge]

not a blocking for testing. open this issue, just want to suggest we add some basic error check in code to this kind of errors. close for now

[q131172019]

PR129 covers this issue.