-
Notifications
You must be signed in to change notification settings - Fork 154
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
wrong host passed to dependencies #337
Comments
Basically, any package with a dependency hosted with another service will fail to install. All the remote prefixes like In my case, installing a github dependency from a bitbucket package, I simply end up with a It is even worse in the case of e.g. a gitlab package with a dependency on a github package, because not only would the |
Effectively reverts r-lib#145. This will break the install of private dependencies for people who used the auth_token (or equivalent) argument. It is however easy and more robust to pass credentials to the deps via environment variables.
The
host
passed at top level is passed to the dependencies. This causes the install to fail when the dependency host should be different.Consider:
Package A on bitbucket. Its
DESCRIPTION
has aRemotes
field withgithub::antoine-sachet/packageB
Package B on github.
Installing package A with
install_bitbucket("bbuser/packageA")
fails!Indeed, with the
host
in...
being passed to the dependencies, I end up with agithub_remote
with host = "api.bitbucket.org/2.0". Incidentally, this results in #322.This issue was introduced by #145 ... It's a bad idea to pass the host to dependencies. I thought the correct way to specify the dependencies' hosts was through the "Remotes" field of the DESCRIPTION?
This is also to some extent a security concern. My credentials are passed to a dependency's host if the corresponding
XXX_remote()
matches the arguments ofinstall_YYY()
.The text was updated successfully, but these errors were encountered: