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.

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

Account invite for user without password doesn't contain password creation link #27914

Closed
keunes opened this issue Jul 11, 2021 · 2 comments
Closed
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug

Comments

@keunes
Copy link

keunes commented Jul 11, 2021

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Go to the 'Users' panel
  2. Add a username & email address
  3. Note that you typed the wrong email address
  4. Correct the email address
  5. Click "Send welcome-email again"

Expected behaviour

The (resent) welcome email contains a tokenised link to set a password, given that the user hasn't set one yet.

Expected, because of this #15770 (comment):

Add user with name and E-Mail: In this case the user receives an E-Mail with a tokenised link to the lostpassword/reset page. This page lets the user set a new password. No change needed.

Actual behaviour

The (resent) welcome email doesn't contain such link.

Server configuration

Operating system: Raspbian GNU/Linux 10 (buster)

Web server: Apache 2

Database: Mariadb

PHP version: PHP 7.3.27-1~deb10u1

Nextcloud version: 20.0.10

Updated from an older Nextcloud/ownCloud or fresh install: updated from older version

Where did you install Nextcloud from:

Nextcloud configuration:

Config report
I{
    "system": {
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trusted_domains": [
            "192.168.xxx.xx",
            "nextcloud.mydomain.eu"
        ],
        "trusted_proxies": "***REMOVED SENSITIVE VALUE***",
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "dbtype": "mysql",
        "version": "20.0.10.1",
        "overwrite.cli.url": "https:\/\/nextcloud.mydomain.eu\/",
        "overwriteprotocol": "https",
        "htaccess.RewriteBase": "\/",
        "dbname": "***REMOVED SENSITIVE VALUE***",
        "dbhost": "***REMOVED SENSITIVE VALUE***",
        "dbport": "",
        "dbtableprefix": "oc_",
        "mysql.utf8mb4": true,
        "dbuser": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "installed": true,
        "memcache.local": "\\OC\\Memcache\\APCu",
        "memcache.locking": "\\OC\\Memcache\\Redis",
        "filelocking.enabled": "true",
        "redis": {
            "host": "***REMOVED SENSITIVE VALUE***",
            "port": 0,
            "dbindex": 0,
            "password": "***REMOVED SENSITIVE VALUE***",
            "timeout": 1.5
        },
        "mail_domain": "***REMOVED SENSITIVE VALUE***",
        "mail_from_address": "***REMOVED SENSITIVE VALUE***",
        "mail_smtpmode": "smtp",
        "mail_sendmailmode": "smtp",
        "mail_smtpauth": 1,
        "mail_smtphost": "***REMOVED SENSITIVE VALUE***",
        "mail_smtpsecure": "ssl",
        "mail_smtpport": "465",
        "mail_smtpname": "***REMOVED SENSITIVE VALUE***",
        "mail_smtppassword": "***REMOVED SENSITIVE VALUE***",
        "maintenance": false,
        "theme": "",
        "loglevel": 2,
        "encryption.legacy_format_support": false,
        "encryption.key_storage_migrated": false
    }
}

Are you using encryption: no

Are you using an external user-backend, if yes which one: no

Logs

Web server error log

Web server error log
Insert your webserver log here

Nextcloud log (data/nextcloud.log)

Nextcloud log
Happy to provide if necessary.
@keunes keunes added 0. Needs triage Pending check for reproducibility or if it fits our roadmap bug labels Jul 11, 2021
@szaimen
Copy link
Contributor

szaimen commented Aug 8, 2021

You misinterpreted something here: #15770 is not yet implemented.
So lets track this in #15770

@szaimen szaimen closed this as completed Aug 8, 2021
@keunes
Copy link
Author

keunes commented Aug 26, 2021

@szaimen The text I quoted (by @ein-giga-self ) says "no change needed". Doesn't this mean that this is already the case on production? Other bullet points list what needs to be implemented still.
Or is this quote only describing what is implemented in a PR that hasn't been merged yet?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug
Projects
None yet
Development

No branches or pull requests

2 participants