Skip to content

Merge dev into main num #2 - #12

Merged
BowMonk merged 2 commits into
mainfrom
dev
Sep 30, 2026
Merged

BowMonk merged 2 commits into
mainfrom
dev

Conversation

@BowMonk

@BowMonk BowMonk commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

Closes PR #11

sdiebolt and others added 2 commits September 30, 2026 14:53
PowerShell treats "?" as a legal variable-name character, so
"$uploadUrl?name=$name" parsed as a single undefined variable and expanded
to "=build_...zip". The upload POST then failed on every matrix leg with
"Invalid URI: The hostname could not be parsed", on both the Windows and
Linux runners.

Expand the variable through braces and check the result is an absolute URI
before posting, so a malformed URI names itself rather than surfacing as a
parser error. The API base is now a parameter so the script can be driven
against a local stub.

Add a regression test covering the upload path, the existing-asset guard,
and the overwrite branch. It fails on the previous interpolation.
fix(ci): build the release asset upload URI correctly
@BowMonk
BowMonk merged commit 126c30f into main Sep 30, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants