-
Notifications
You must be signed in to change notification settings - Fork 64
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
Fix hard coded Sonatype URL #215
Conversation
@steinybot Thanks. I've modified the PR to simply the setting for users. |
> | ||
> ```scala | ||
> // For all Sonatype accounts created on or after February 2021 | ||
> ThisBuild / sonatypeCredentialHost := "s01.oss.sonatype.org" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be:
> ThisBuild / sonatypeCredentialHost := "s01.oss.sonatype.org" | |
> ThisBuild / sonatypeCredentialHost := Sonatype.sonatype01 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@xerial I was a bit too late with this comment but FYI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah. I added this constant because I thought it can be used like that, but using "s01.oss.sonatype.org" would be more straightforward for users.
This fixes the workaround for #214 although doesn't fix that issue completely.
In order to support both Sonatype URLs I think this ought to first try
oss.sonatype.org
and then fallback tos01.oss.sonatype.org
.In any case it should still be possible to override the URL manually which this PR enables.