Skip to content

webui: Fix login issue#591

Merged
frb121 merged 1 commit intomasterfrom
dev/fbergkemper/master/s4269-master
Sep 25, 2020
Merged

webui: Fix login issue#591
frb121 merged 1 commit intomasterfrom
dev/fbergkemper/master/s4269-master

Conversation

@frb121
Copy link
Contributor

@frb121 frb121 commented Sep 10, 2020

Remove a misplaced return null in AuthController.

Fixes #1039: Can not login in webui

@frb121 frb121 force-pushed the dev/fbergkemper/master/s4269-master branch 2 times, most recently from f048318 to bcb5f4d Compare September 10, 2020 08:52
@frb121 frb121 force-pushed the dev/fbergkemper/master/s4269-master branch from bcb5f4d to 9eee57d Compare September 14, 2020 17:29
@frb121 frb121 force-pushed the dev/fbergkemper/master/s4269-master branch from 9eee57d to 1730cd7 Compare September 15, 2020 08:54
This commit fixes a login issue which occurred when the update
information could not be retrieved.

- Restructured AuthController for readability and optimization
- Use the ZF2 Session Container instead of $_SESSION
- Introduce AuthController method checkAPIStatusDIRD
- Introduce AuthController method checkACLStatusDIRD
- Introduce AuthController method checkUpdateStatusDIRD

The commit also fixes a minor issue with the UpdateAlert ViewHelper.

Fixes #1039: Can not login in webui
@frb121 frb121 force-pushed the dev/fbergkemper/master/s4269-master branch from 1730cd7 to 0691180 Compare September 24, 2020 14:15
@frb121 frb121 changed the title webui: Fix occasional login problems webui: Fix login issue Sep 24, 2020
Copy link
Member

@arogge arogge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@frb121 frb121 merged commit 2f25fba into master Sep 25, 2020
alaaeddineelamri pushed a commit to alaaeddineelamri/bareos that referenced this pull request Sep 30, 2020
@frb121 frb121 deleted the dev/fbergkemper/master/s4269-master branch October 30, 2020 11:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants