Bug #2828
Wrong/empty password for oVirt leads to an exception
Description
Instead of nice red bubble, 401 nasty error is thrown.
Also, I was somehow able to create a compute resource with wrong password and it saved, but when I click on edit, 401 error. Not able to recover through the UI.
We should:
- print nice error when 401
- check username/password when saving (maybe this is generic issue)
Associated revisions
History
#1
Updated by Lukas Zapletal over 9 years ago
Updated by - Description updated (diff)
- Target version deleted (
1.3.0)
#2
Updated by The Foreman Bot about 5 years ago
Updated by - Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/5176 added
#3
Updated by Ido Kanner about 5 years ago
Updated by - Assignee set to Ido Kanner
#4
Updated by Tomer Brisker almost 5 years ago
Updated by - Legacy Backlogs Release (now unused) set to 330
#5
Updated by Ido Kanner almost 5 years ago
- % Done changed from 0 to 100
- Status changed from Ready For Testing to Closed
Applied in changeset 113ec27812f5f639b9a2b22e80f483834688c5bf.
Fixes #2828 - Report an error when trying to save ovirt resource
On both create and update, there was no exception handler to captured
given exception. That resulted in having an exception screen instead of
an error message.
The fix capture all of the given exceptions, and report them as an
error.
added capture for Exception
added capture for StandardError, after a comment i got from gh bot