Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Existing passwords blank for new users #4459

Open
edbourne opened this issue Nov 13, 2024 · 3 comments
Open

Existing passwords blank for new users #4459

edbourne opened this issue Nov 13, 2024 · 3 comments

Comments

@edbourne
Copy link

Steps to reproduce

  1. Create new user
  2. View an existing password - existing passwords are blank for new user

Expected behaviour

Existing passwords should be visible to new user

Actual behaviour

All existing passwords are blank when viewing as the new user. Creating a new password (as the new user) works as expected, the new user and existing users can see it.
Also, creating a new password as an existing user (after the new user is created) works as expected, both existing users and the new user can see these.

Existing users behave normally and can see all passwords that they normally can. All existing user passwords have been re-encrypted to support the new Symfony library.

The new user is in the same roles as other existing users, so should be able to see the same passwords.
Have tried re-running the upgrade and no errors are reported. Not certain when this started I'm afraid as I've not tried to create a new user for a year or so...

Any help much appreciated!

Cheers - Ed

Server configuration

Operating system: OpenSUSE Leap 15.5

Web server: Apache2 2.4.51

Database: MariaDB 10.6.18

PHP version: 8.1.29

Teampass version: latest from Git as of today

Teampass configuration file: Not certain which file is required here - tp.config.php seems to have moved into the DB?

Updated from an older Teampass or fresh install: updated, instance has been active since 2.xx days
PLEASE attach to this issue the file /includes/config/tp.config.php.

Client configuration

Browser: Firefox (issue also present in Chrome & Edge)

Operating system: Windows 11

Logs

Web server error log

No errors logged in apache error log

Log from the web-browser developer console (CTRL + SHIFT + i)

This page is in Almost Standards Mode. Page layout may be impacted. For Standards Mode use “<!DOCTYPE html>”.
[index.php](https://swt-ut.switalskis.com/teampass/index.php?page=items&loginForm)
Cookie “teampass_theme” does not have a proper “SameSite” attribute value. Soon, cookies without the “SameSite” attribute or with an invalid value will be treated as “Lax”. This means that the cookie will no longer be sent in third-party contexts. If your application depends on this cookie being available in such contexts, please add the “SameSite=None“ attribute to it. To know more about the “SameSite“ attribute, read https://developer.mozilla.org/docs/Web/HTTP/Headers/Set-Cookie/SameSite [index.php](https://swt-ut.switalskis.com/teampass/index.php?page=items&loginForm)
​``
@corentin-soriano
Copy link
Contributor

Look like your user does not have any sharekey for this item.
Do you have any pending task? (to be verified with an administrator account)

@edbourne
Copy link
Author

Nope, no pending tasks. ("No matching records found")
image

This is the task scheduler settings:
image

(not sure if the maintenance operation schedule is optimal - none of those were originally scheduled to run automatically, I picked some likely-sounding times for them. They have also all been run manually)

Cheers

@Torx999
Copy link

Torx999 commented Nov 14, 2024

we have also the same problem. solution till now - old user who can see the "old" password creates new item and moves password there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Torx999 @corentin-soriano @edbourne