mirror of
https://github.com/cypht-org/cypht.git
synced 2026-04-26 05:26:00 +03:00
[GH-ISSUE #432] Not loading settings or mail until logout and login is done #358
Labels
No labels
2fa
I18N
PGP
Security
Security
account
advanced_search
advanced_search
announcement
api_login
authentication
awaiting feedback
blocker
bug
bug
bug
calendar
config
contacts
core
core
devops
docker
docs
duplicate
dynamic_login
enhancement
epic
feature
feeds
framework
github
github
gmail_contacts
good first issue
help wanted
history
history
imap
imap_folders
inline_message
installation
keyboard_shortcuts
keyboard_shortcuts
ldap_contacts
mobile
need-ssh-access
new module set
nux
pop3
profiles
pull-request
question
refactor
release
research
saved_searches
smtp
strategic
tags
tests
themes
website
wordpress
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/cypht#358
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @kk6mrp on GitHub (Sep 18, 2020).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/432
Originally assigned to: @jasonmunro on GitHub.
🐛 Bugreport
Likely there is something I am doing wrong here but I can't seem to find the issue. When opening Cypht in a new tab or browser window at the first login it loads the main Cypht page however it says, "You don't have any data sources assigned to this page." and my user settings are not loaded. After logging out and logging back in, I see data sources and settings loaded like normal. Here is the log files for after the first login:
I did notice after the first login there are two cookies, hm_id and hm_session. After logging out I see four cookies, hm_id, hm_msgs, hm_reload_folders, and hm_session. After logging in again there are two cookies, hm_id and hm_session.
If I have logged out and logged back in I can open more new tabs and load Cypht. If I have logged in and back out and then open a new tab to log in the third time, I have to repeat the three step again in the new tab. Hopefully I worded that well enough to be understood.
Version & Environment
Rev: [
ed035eea]OS: [Ubuntu Server 18.04.5 LTS]
Browser: Tested in Firefox 78ESR and Chromium 85
Steps to reproduce
@jasonmunro commented on GitHub (Sep 18, 2020):
I don't understand exactly. This only happens if you have a browser open and open a new tab/window to login to Cypht? Or does it happen anytime you try to login?
@kk6mrp commented on GitHub (Sep 18, 2020):
I'm sorry about the confusing description, this happens any time I try to log in.
@kk6mrp commented on GitHub (Jan 19, 2021):
@jasonmunro Do you have any suggestions for how to find the issue here or where to look? Logging out and back in to properly log in is a little frustrating.
@jasonmunro commented on GitHub (Jan 20, 2021):
@kk6mrp Sorry, I'm at a bit of a loss about how this could be happening. Here is what I would try to eliminate some possible causes:
Don't forget when you change your hm3.ini file you need to rerun the config_gen.php script. If you can try those things and let me know if anything works it would help move this forward. Thanks!
@kk6mrp commented on GitHub (Jan 20, 2021):
So this is on the latest master branch:
@kk6mrp commented on GitHub (Jan 27, 2021):
@jasonmunro I also found that changing session handling from DB to PHP does not retain my session if I click "Stay Logged In"
Do you think this could have anything to do with me using SQLite?
@kk6mrp commented on GitHub (Jun 8, 2021):
I've reinstalled Cypht on a new server and cannot reproduce this issue, feel free to reopen if needed!
@kk6mrp commented on GitHub (Jun 12, 2021):
This issue came back after I set the "First page after login" in the Site Settings and realized that it stems from the change you made for me on
github.com/jasonmunro/cypht@ad8b3993ccI didn't notice the correlation at the time but now I see it.I'm not sure if it is the idle timer interfering with the
Hm_Dispatch::page_redirectas discovered here https://github.com/jasonmunro/cypht/issues/273#issuecomment-398881553 or if theHm_Dispatch::page_redirectis being called at an unexpected time.Pull request #492 should safely close this issue.
@jasonmunro commented on GitHub (Jun 12, 2021):
Thanks for this and great catch. Merging into master + 1.3.0 in just a minute
@kk6mrp commented on GitHub (Jun 12, 2021):
Do you know what the root cause is?
@jasonmunro commented on GitHub (Jun 12, 2021):
I believe the issue is that it is redirecting before the session is saved, so you don't stay logged in. When you login the second time you are already at the correct url so no redirect happens and you don't get logged out. Regardless this change is the "right" way to do redirects from this context so it is better regardless.
@kk6mrp commented on GitHub (Jun 12, 2021):
Ok, I'm not very familiar with the codebase yet but that makes sense... Thanks!