| Summary: | login: An error occurs when authentication fails | ||
|---|---|---|---|
| Product: | [Retired] oVirt | Reporter: | Asaf Shakarchi <asaf> |
| Component: | ovirt-engine-webadmin | Assignee: | Vojtech Szocs <vszocs> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | unspecified | CC: | ecohen, iheim, ykaul |
| Target Milestone: | m2 | ||
| Target Release: | 3.1 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2012-08-09 07:59:05 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Bug Depends On: | |||
| Bug Blocks: | 719977 | ||
|
Description
Asaf Shakarchi
2011-07-04 12:41:16 UTC
If this error is unexpected, it must be solved, right now it wont be showed due to this fix: LoginSectionPresenter onReveal() disables FrontendEventsHandlerImpl-related error popups, onHide() enables them again NOTE: Commit: 57a1daa42272cd833015d8c977bf2b9234600333 hiding errors when login popup is revealed, There are multiple calls within UiCommon to Frontend.RunQuery() early on, even when the user is not logged in yet. There are two UiCommon integration classes within WebAdmin, which use ErrorPopupManager to show an error popup upon some error event: - FrontendEventsHandlerImpl - FrontendFailureEventListener Proposed solution: inside LoginSectionPresenter.onReveal(), tell ErrorPopupManager to suppress showing popups [currently this is done only for FrontendEventsHandlerImpl]. In any case, UiCommon developers should state how the application should react in this situation (application is launched and login form is shown, user is not logged in yet). closing ON_QA bugs as oVirt 3.1 was released: http://www.ovirt.org/get-ovirt/ |