Skip to content

fix: header must be a string, not array#953

Merged
fgreinacher merged 1 commit intosemantic-release:masterfrom
candrews:header-string
Feb 24, 2026
Merged

fix: header must be a string, not array#953
fgreinacher merged 1 commit intosemantic-release:masterfrom
candrews:header-string

Conversation

@candrews
Copy link
Contributor

@candrews candrews commented Feb 23, 2026

The header must be a string, not an array. Per the got documentation, headers is of type object<string, string>. See https://github.com/sindresorhus/got/blob/v14.6.6/documentation/2-options.md#headers

This PR fixes the mis-typing problem introduced in #919 which appears to cause GitLab releases to fail with incorrect EINVALIDGLTOKEN Invalid GitLab token. errors.

@arvest-bjoneson @fgreinacher

@fgreinacher
Copy link
Contributor

fgreinacher commented Feb 23, 2026

@candrews The diff looks off. Please double check and ping me when ready for review.

The header must be a string, not an array. Per the got documentation, `headers` is of type `object<string, string>`. See https://github.com/sindresorhus/got/blob/v14.6.6/documentation/2-options.md#headers

Fixes the mis-typing problem introduced in semantic-release#919 which appears to cause GitLab releases to fail with incorrect `EINVALIDGLTOKEN Invalid GitLab token.` errors
@candrews
Copy link
Contributor Author

candrews commented Feb 23, 2026

@candrews The diff looks off. Please double check and ping me when ready for review.

It's fixed. All tests pass, lint reports success as well.

@candrews
Copy link
Contributor Author

candrews commented Feb 24, 2026

ping me when ready for review.

@fgreinacher pinging as requested, thank you

@fgreinacher fgreinacher merged commit dea1c55 into semantic-release:master Feb 24, 2026
14 checks passed
@fgreinacher
Copy link
Contributor

fgreinacher commented Feb 24, 2026

Thanks for the fix!

@github-actions
Copy link

🎉 This PR is included in version 13.3.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sathieu
Copy link

sathieu commented Feb 25, 2026

@fgreinacher Please rollback this change. It is breaking token authentication (see #954, #955 and #956).

@fgreinacher
Copy link
Contributor

fgreinacher commented Feb 25, 2026

@fgreinacher Please rollback this change. It is breaking token authentication (see #954, #955 and #956).

Thanks for the ping and sorry for not having verified this enough!

Revert PR is up at #957 and will be merged asap.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants