-
Notifications
You must be signed in to change notification settings - Fork 8
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
The CI step indicated in the readme doesn't work #28
Comments
Thanks for the report, an easy workaround could be using |
After a bit of experimentation, it looks like Cargo happily passes through
|
Actually, scratch all of that. We expect the password to be in the username so modifications would be needed for the above. Maybe |
Got it to work with that:
It's a bit annoying because it requires a key that would be valid as authentication method but it works. |
Note that this is not actually a solution if you need to publish a package depending on another package in a custom registry. The changed URL will be saved by cargo into the package metadata and it'll completely break down the line as you'll have packages coming from |
Second note, you don't need to do the |
Cargo doesn't allow passwords in custom registry URLs (rust-lang/cargo#6242). Even though
ssh
doesn't allow passwords in its URIs, cargo still sees it as one (it's been the case since at least 1.40 which is the oldest version I tested).The text was updated successfully, but these errors were encountered: