helm / chart-releaser

Hosting Helm Charts via GitHub Pages and Releases
Apache License 2.0
684 stars 113 forks source link

fix: allow package versions with extended semversioning #405

Closed jlpedrosa closed 6 months ago

jlpedrosa commented 6 months ago

It is reported that chart-releaser fails when the version number has some separator: issue

When a tag has the + symbol ie: 2.0.0+Chart1, which is semver compliant see docs.

When githubs return the download URL for the chart, it contains escape character %2B, but the actual filename is +Chart1. This causes the log/trace linked in the issue above.

This PR adds the filenameParameter to the addToIndexFile function. So instead of receiving the URL only, it receives the url and the correct filename for the asset, that later parts of the code will try to open.

Also this PR adds support for ssh url syntax for remotes, so it also can be executed locally.