fix (cmd) : URL Port Corrupts Input in Import-URL Command

[aniket866]

Description

The import-url command parses arguments formatted as url:primary:secret. It uses strings.Split(f, ":") and reconstructs the URL by concatenating the first two elements. If the URL contains a port number (e.g. http://localhost:8585/spec.yaml), the split splits it into ["http", "//localhost", "8585/spec.yaml"]. The code then drops the port and path, changing the URL to http://localhost.

Reason Why It Is Valid

It breaks importing artifacts from local test instances or any custom web server that runs on a non-default port (containing a colon).

Bug Simulation Workflow

graph TD
    A["User runs: microcks import-url http://localhost:8585/api.yaml"] --> B["Split by ':' produces http, //localhost, 8585/api.yaml"]
    B --> C["Reconstruction: urlAndMainAtrifactAndSecretName[0] + ':' + urlAndMainAtrifactAndSecretName[1]"]
    C --> D["Resulting URL is http://localhost"]
    D --> E["API import request sent to http://localhost instead of http://localhost:8585/api.yaml"]

Loading

after fix:

• importURL.go

Replace the simple : splitting logic with a more robust parser that checks segments from the right end of the string to identify the optional main and secret parameters.

• [NEW]
  importURL_test.go

Add unit tests to verify the parsing logic for various URL combinations, including:

• Standard URLs (no port, no suffixes)
• URLs with ports (no suffixes)
• URLs with ports and main suffix
• URLs with ports, main suffix, and secret suffix
• Standard URLs with main and secret suffixes

Closes #463
@lbroudoux @Harsh4902
do checkout whenever you get time

[aniket866]

hi @Harsh4902 do checkout this one , whenever you get time

[Vaishnav88sk]

Since #460 also touches cmd/importURL.go, please rebase/resolve the overlap so we keep a single parsing implementation in this file.

Otherwise, LGTM.

[aniket866]

Since #460 also touches cmd/importURL.go, please rebase/resolve the overlap so we keep a single parsing implementation in this file.

Otherwise, LGTM.

@Vaishnav88sk
done rebased and resolved,

[Vaishnav88sk]

The original import-url parsing fix looks good now, and the overlap with #460 appears resolved in the parsing logic itself.

However, this PR is no longer scoped to that fix only. It now includes several unrelated changes as well, including:

• CreateTestResult error-handling changes
• logout named context changes
• OAuth token/log redaction changes
• other parsing/default-handling changes
• merge-from-master noise

Please keep this PR focused on the import-url port/parsing fix only (plus its direct tests).

[Vaishnav88sk]

Please pass the DCO.

[aniket866]

Please pass the DCO.

done

[Vaishnav88sk]

LGTM
@Harsh4902