Skip to content

Adjust bintray resolver credentials to honor realm values#514

Merged
fsat merged 1 commit intotypesafehub:masterfrom
longshorej:fix-bintray-realm
Jun 28, 2017
Merged

Adjust bintray resolver credentials to honor realm values#514
fsat merged 1 commit intotypesafehub:masterfrom
longshorej:fix-bintray-realm

Conversation

@longshorej
Copy link
Copy Markdown
Contributor

This PR ensures that the Bintray resolver only uses credentials if realm is omitted or equal to Bintray. Values following other realm declarations are omitted.

Fixes #507

Manual Test

Ensure that the test below fails on master, then rerun and ensure it passes with this PR.

$ rm /home/longshorej/.conductr/cache/bundle/visualizer-2.1.0-cabaae7cf37b1cf99b3861515cd5e77a16fa9638e225fa234929cc1d46dde937.zip
-> 0
$ cat ~/.lightbend/commercial.credentials 
realm = Bintray
host = dl.bintray.com
user = <redacted>
password = <redacted>
realm = Bintray API Realm
host = api.bintray.com
user = myuser
password = mypassword

-> 0

@fsat fsat merged commit b2a8d53 into typesafehub:master Jun 28, 2017
@fsat fsat mentioned this pull request Jun 29, 2017
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.

Credentials file can only contain one section

2 participants