Skip to content
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

Add support for netrc in jvm_maven_import_external #1509

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

dmivankov
Copy link
Contributor

Description

auth= parameter for repository_ctx.download is both required to fetch from private registries and is not well-documented bazelbuild/bazel#13709 (comment)

With this change

rules_scala_toolchain_deps_repositories(
    maven_servers = ["some_private_artifactory_url"]
)

would be able to authenticate using default .netrc (like ~/.netrc)

Motivation

Private maven registry could be useful to host patched versions of libraries or to ensure that there's a private persistent mirror for those artifacts

`auth=` parameter for `repository_ctx.download` is both
required to fetch from private registries and is not well-documented
bazelbuild/bazel#13709 (comment)

With this change
```python
rules_scala_toolchain_deps_repositories(
    maven_servers = ["some_private_artifactory_url"]
)
```
would be able to authenticate using default .netrc (like `~/.netrc`)
Copy link
Collaborator

@liucijus liucijus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @dmivankov!

@simuons simuons merged commit 0391ef4 into bazelbuild:master Oct 2, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants