search

rh-messaging / cli-proton-python

Proton Python client provided by QE for testing.
Apache License 2.0
4 stars 6 forks source link

FIX(ErrorsHandler): Zero error-code was returned when user was not va… #40

Closed rkubis closed 6 years ago

rkubis commented 6 years ago

…lidated on broker

~~ Cause: Unhandled case when username and/or password is missing in broker URL, for example: user:@host:port/address. Consequence: It looked that client ended without error. Fix: Raise exception when user is not validated on broker side. Result: Exception is raised and client ends with non-zero code. ~~ Relates: Closes #39 issue

rkubis commented 6 years ago
  1. Unnecessary brackets are everywhere in this file, it was my template for change, but I can remove them.
  2. OK, I can change it to error code "AMQ119031".
rkubis commented 6 years ago

Fix updated after review.