-
Notifications
You must be signed in to change notification settings - Fork 157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Failed upload leaks credentials #85
Comments
Signed-off-by: bndw <benjamindwoodward@gmail.com>
Fixes github-release#85 Signed-off-by: bndw <benjamindwoodward@gmail.com>
In addition to this, the access token should not be in the URL as it is deprecated. Github sent me a message about this today. Since the project looks dead, looks like I'll be using one of the forks and if they don't fix this, I'll be forking the fork.. |
@jc21 did you fork the fork? I need this fix as well :) |
I thought I fixed this... is it not fixed? |
It's worth noting the official gh tool supports release management now, depending on your situation, switching to the official tool may be a good option now. In my case, I found https://cli.github.com/manual/gh_release_create a perfectly serviceable replacement. |
@ocdtrekkie I just noticed that myself! So disregard my comments here, I'll use their tool instead. |
apologies for the delay; I've just merged the fix |
First, thanks for providing a simple tool for working with Github releases!
It appears that when using the
upload
command to a non-existent repository, theaccess_token
is leaked in the error message. This is a problem when running this tool in an open-source project's CI system.The text was updated successfully, but these errors were encountered: