-
-
Notifications
You must be signed in to change notification settings - Fork 138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
InvalidToken error at login for v2.36.1+ with "Custom Login Mapping" extension #1706
Comments
The POST Action "Login" reponse data should also have a Can you check that? If it is different, i might know where the problem is. |
Hi, sorry I didn't mean to not respond. Life got in the way! I notice that I can login with a test account, but not my main account. My main account has 2 extra accounts configured. Could that be the problem? I'll check if I can remove these accounts from my test setup. |
Hi, should adding the 1 line change from daf02e5 fix the issue? |
Just adding the line should be sufficient. |
The manual edit didn't seem to work, but I updated to v2.37.3 and this build works without issues, also on logins with multiple accounts. Great! Too bad I can't donate on GitHub, so I donated via PayPal :) |
I spoke to soon I guess. I now get "Folders error: Call to a member function Email() on null". |
Even if I remove the entire PS: I tested the update on my test install and I thought the login went find but I didn't correctly used my test URL which redirected my to my production version so I thought it worked and updated my production too. Woops. |
This is the log now:
|
Oh great! Now we atleast have progress with a different error. Thanks for the donation. |
Ok the error resolved. |
Reopened as token mismatch still occurs for ervee |
As #1754 is closed. I'll just post here that there is another user without multiple accounts configured with login problems. I just got a message from another user with a "Folders error: HTTP Token mismatch" and not being able to login. logs:
Not sure if this is related. I can create a new bug for it if you want. I could work-around the update to 2.38 for myself, but this user relies on webmail so I had to revert back to 2.33 (known good for me). Given the security announcement I hope you can get me on 2.38 back soon. Please ask if you need anything from me. |
I don't get it. So v2.34 should also work on your instance. Please check each version until it fails. Secondly, v2.38 should provide way more log info then "Crypt[WARNING]: Decrypt() invalid $data or $key". Maybe:
|
I'll get my test setup on 2.33 and upgrade upward from that. It is very possible I did skip a couple of versions. So I'll get back on that. I'll also enable debug on the test setup. Answers:
|
I have iterated trough all updates and this is what I get: v2.33.0 works This is a debug log from v2.36.2 where the problem started:
And a debug log from v2.38.0 (with ERROR->ERR fix):
|
Thanks for your amazing test result!
This means that So maybe |
I did some some testing (more logins/logouts) around version v2.36.2 and I'm able to break it now with v2.36.1 too. Perhaps I was too fast with my test methodology. After changing from v2.36.0 to v2.36.1 and logging in/out a couple of times with a test account and my own account suddenly the Password" box changes to "Passphrase" and then it fails. I retested it all with changing the "APP_VERSION" in index.php and then restarting nginx and php8.1-fpm and that helps. I can now consistently break and fix it when switching between v2.36.0 and v2.36.1. I got the changes from 16cc17a in Account.php to my v2.38.0 folder, set this version in index.php, restarted nginx and php8.1-fpm, refreshed and logged in with my test user (works) and then my own user and this is the log produced:
|
v2.36.1 uses SensitiveString and a different So maybe "Custom Login Mapping" extension is failing. |
My server does not use the plugin.
But yours is probably different and uses the extension. |
I've just tested the new v2.3 Custom Login Mapping like this: This changed the login for imap to be I've updated the extensions README with: So basically speaking: So when you login as USER, it will be lowercase like |
In my custom login mapping I have: So I never used a secon colon (:). Is that mandatory? My setup with the plugin is as follows: I have "Use short login" and "Lowercase login" enabled on both IMAP and SMTP domain settings. |
Aha, we are on to something. |
Did you manage to get anywhere with this? |
It seems to be an issue with domain resolving. Do you have Admin -> Domains -> |
Yes I have a domain.tld for the user logging in with the email address. But maybe I found the issue... When investigating another issue I was drawn to the Now changing the whitelist is not that hard so I'll just add both for now :) Is this an intended change? Or did some check before the plugin got moved to after the plugin or something? |
Aha! That explains why i could not replicate the issue. |
Fixed the logging and added a missing one on account switching. Yes the whitelist was "fixed" as it has nothing to do with imap login but with email addresses. |
Okay. Then this is fixed. Using a name not in the whitelist result in the same nondescript error so I hope you can fix that if you have not already. |
Describe the bug
After update to 2.37.2 over a known-good working 2.33.0 test install I get an [object Object] error at login and get thrown back at the login screen.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
See my mailbox :)
Screenshots

Please complete the following information:
Debug/logging information
Read here how to log
Additional context
None
The text was updated successfully, but these errors were encountered: