[BUG] in command update the variable "host" is not assigned.

[desnudaenlaplaya]

Describe the bug

The variable "host" is not assigned.

In https://github.com/OpenCHAMI/magellan/blob/main/cmd/update.go#L67

To Reproduce
Steps to reproduce the behavior:

1. just call the update command and the empty url is raised:
   $ ./magellan update https://:443
   --username root --password 0penBmc
   --firmware-url http://127.0.0.1:1337/obmc-phosphor-image-.static.mtd.tar
   --component BMC
   {"level":"error","error":"failed to parse URI: parse "": empty url","time":"2025-01-29T17:25:35+01:00","message":"failed to update firmware"}

Expected behavior
Just work if is valid

Fix

$ git diff cmd/update.go
@@ -57,6 +58,9 @@ var updateCmd = &cobra.Command{
                                return
                        }
+                       host = arg
                        // initiate a remote update
                        err := magellan.UpdateFirmwareRemote(&magellan.UpdateParams{
                                FirmwarePath:     firmwareUrl,

[davidallendj]

Does this still work correctly when changing this line to URL: arg instead?
https://github.com/OpenCHAMI/magellan/blob/main/cmd/update.go#L67

I think that line was supposed to be similar to the one above on line 48:
https://github.com/OpenCHAMI/magellan/blob/main/cmd/update.go#L48

[alexlovelltroy]

Looks like there's a fix in the linked branch. Do we have a test to validate that? I'd love to get that closed.