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.

Sign up for GitHub

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

Jump to bottom

conan upload dumps bintray login page html as text #1832

Closed
VestniK opened this issue Sep 30, 2017 · 5 comments
Closed

conan upload dumps bintray login page html as text #1832

VestniK opened this issue Sep 30, 2017 · 5 comments
Assignees
@lasote
Milestone
0.28

Comments

@VestniK
Copy link

VestniK commented Sep 30, 2017

I've created bintray OSS account, group and added repository to the group. But I've used wrong URL while adding remote to conan (I've copied group URL instead of repository URL into my conan remote add command). When I've tried to upload package I've got dump of bintray login page instead of usable error. It tooks qiute some time to investigate what's going on in such situation. Can you improve error reporting for a case when repository URL points to some resource which is not conan repository?
@memsharded
Copy link
Member

Yes, could make sense, though sometimes the html provided by the server also is informative, like a message from a proxy server. Lets have a look.

@VestniK
Copy link
Author

VestniK commented Oct 3, 2017

May be you can separate dump of the response from the error message with an empty line so they will be visually distinguishable.

@memsharded
Copy link
Member

Could you please copy here your URL and the output message? I am trying to reproduce, but so far I can only see messages like ERROR: No JSON object could be decoded, but no login page dumped.

Also, please, make sure you are using latest conan 0.27. Thanks!

@VestniK
Copy link
Author

VestniK commented Oct 4, 2017

Yes I can reproduce it with 0.27:

vestnik@uk-rnd-104:~/Development/VestniK/conan_deps$ conan --version
Conan version 0.27.0
vestnik@uk-rnd-104:~/Development/VestniK/conan_deps$ conan remote add test https://bintray.com/vestnik/VestniK
vestnik@uk-rnd-104:~/Development/VestniK/conan_deps$ conan user -p *** -r test sirvestnik   
Change 'test' user from None (anonymous) to sirvestnik
vestnik@uk-rnd-104:~/Development/VestniK/conan_deps$ conan upload '*@VestniK/stable' -c -r test --all
ERROR: Invalid header value 'Bearer \n<!doctype html>\n<!--[if IE 9]><html class="lt-ie10" lang="en" >login page dump here</html>\n'

@lasote lasote self-assigned this Oct 4, 2017
@lasote lasote added this to the 0.28 milestone Oct 4, 2017
@lasote lasote mentioned this issue Oct 4, 2017
Merged
@memsharded
Copy link
Member

Should be fixed now in 0.28, please try.

@pvicente pvicente added the fixed label Feb 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lasote lasote

Labels
None yet
Projects
None yet
Milestone
0.28
Development

No branches or pull requests
4 participants
@pvicente @VestniK @lasote @memsharded