fix: Give an explicit message when the server respond with an error #317

 (#334)

* fix empty API url

* fix: Give an explicit message when the server respond with an error #317

* update release.md

cmd/shared.go

@@ -62,7 +62,7 @@ func (app *SharedFlags) InitSharedFlags() {
 func (app *SharedFlags) SetFlags(fs *flag.FlagSet) {
 	fs.StringVar(&app.ConfigurationFile, "use-configuration", app.ConfigurationFile, "Specifies the configuration to use")
 	fs.StringVar(&app.Server, "server", app.Server, "Immich server address (http://<your-ip>:2283 or https://<your-domain>)")
-	fs.StringVar(&app.API, "api", "", "Immich api endpoint (http://container_ip:3301)")
+	fs.StringVar(&app.API, "api", app.API, "Immich api endpoint (http://container_ip:3301)")
 	fs.StringVar(&app.Key, "key", app.Key, "API Key")
 	fs.StringVar(&app.DeviceUUID, "device-uuid", app.DeviceUUID, "Set a device UUID")
 	fs.StringVar(&app.LogLevel, "log-level", app.LogLevel, "Log level (DEBUG|INFO|WARN|ERROR), default INFO")

docs/releases.md

@@ -1,5 +1,12 @@
 # Release notes 
 
+### fix: [#317](https://github.com/simulot/immich-go/issues/317)
+When the server doesn't reply as expected, the message is now explicit:
+```
+the ping API end point doesn't respond at this address: http://localhost:2283/api/server-info/ping
+```
+
+
 ### feat: Server's activity graph
 The UI mode now show the current activity of the immich server. After 10 seconds of zero activity, the program stops
 

immich/client.go

@@ -109,7 +109,7 @@ func (ic *ImmichClient) PingServer(ctx context.Context) error {
 	r := PingResponse{}
 	err := ic.newServerCall(ctx, "PingServer").do(get("/server-info/ping", setAcceptJSON()), responseJSON(&r))
 	if err != nil {
-		return err
+		return fmt.Errorf("the ping API end point doesn't respond at this address: %s", ic.endPoint+"/server-info/ping")
 	}
 	if r.Res != "pong" {
 		return fmt.Errorf("incorrect ping response: %s", r.Res)