Conan: [UX] Get rid of traceback output on `authenticate` error

Created on 30 May 2019  路  5Comments  路  Source: conan-io/conan

To help us debug your issue please explain:
Conan Version: 1.15.1

Hi!

Since last weekend I've updated some packages to support Clang 8 and MSVC 16, when requested by any user. However, OpenSSL is failing when trying to upload MSVC 16:

These logs are from today:

https://ci.appveyor.com/project/ConanCIintegration/conan-openssl/builds/24895637/job/x7697tjf26k60kbd#L5356
https://ci.appveyor.com/project/ConanCIintegration/conan-openssl/builds/24895637/job/nvq8ey2e4rtglqfq#L7449

And there are more from last weekend.

Or Bintray is unstable now, or something is not working when multiple packages are being uploaded at same time. Could we avoid this error using some retry mechanism?

  • [x] I've read the CONTRIBUTING guide.
  • [x] I've specified the Conan version, operating system version and any tool that can be relevant.
  • [x] I've explained the steps to reproduce the error or the motivation/use case of the question/suggestion.
low ux good first issue medium queue bug

All 5 comments

Hi! conan upload... already have retry arguments, maybe you should consider using them in conan-package-tools... about the bintray issue, I will tag it, but usually those are temporary problems very hard to trace, reproduce and solve 馃槥

Following the stack trace, actually the error occurs before uploading, when authenticating ...

CPT by default tries to upload 3x, but there is no mechanism preventing login retry on API. So I see two options:

  • Wrapping authenticate step in CPT with try except, retrying when necessary
  • Add retry parameter to conan_api.authenticate

You are right, @uilianries, it is about the conan_api.authenticate function. I also agree with the PR, I think the retry should go in CPT, there is no need for an additional argument in Conan CLI to retry this operation.

Anyway, I'll keep this open and labeled as bug because we need to get rid of the nasty traceback and output a meaningful error to the user.

Is this still a real issue? I cannot checks the linked appveyor to see the stacktrace generated. @jgsogo could you provide more info to reproduce/fix?

Now I get a meaninful error:

Remote 'jgsogo' username: jgsogo
Please enter a password for "jgsogo" account: 
ERROR: Wrong user or password
Remote 'jgsogo' username: jgsogo
Please enter a password for "jgsogo" account: 
ERROR: Wrong user or password
Remote 'jgsogo' username: jgsgo
Please enter a password for "jgsgo" account: 
ERROR: Wrong user or password
ERROR: Too many failed login attempts, bye!. [Remote: jgsogo]

We might have already fixed it, I'll close the issue while there is no evidence.

Was this page helpful?
0 / 5 - 0 ratings