Modify

Opened 15 years ago

Closed 5 years ago

#6318 closed defect (worksforme)

[patch] Users can't get verification email

Reported by: rue@… Owned by:
Priority: low Component: AccountManagerPlugin
Severity: normal Keywords: needinfo user register verifiy
Cc: Trac Release: 0.11

Description

I've installed the plugins and enable the all function in this plugin.Once I create new account , users can't get the verification mails. Have you met this situation?

Thanks,

Attachments (1)

acctmng.email.patch (4.1 KB) - added by larsivi 14 years ago.
Patch to solve this issue

Download all attachments as: .zip

Change History (19)

comment:1 Changed 15 years ago by Michael Renzmann

Make sure you have the following in your trac.ini:

[notification]
smtp_enabled = true

comment:2 Changed 15 years ago by anonymous

What is confusing is that AccountChangeListener has to be enabled in order to send verification mails. The log output won’t tell much and it should exist a warning or even an error message that it isn’t possible to send verification mails when certain dependencies aren’t met.

Changed 14 years ago by larsivi

Attachment: acctmng.email.patch added

Patch to solve this issue

comment:3 Changed 14 years ago by larsivi

I have attached a patch that fixes this issue (or at least it fixes how I experienced it). What happens is that the notification code assumes that the username is an email address, even if an email explicitly has been set (in fact nothing is sent if an email isn't explicitly set).

This does mean changes to the API, but totally sane ones IMO.

comment:4 Changed 14 years ago by larsivi

I have created http://bitbucket.org/larsivi/trac-accountmanager-plugin/ as an alternative place to get this plugin and any patches I've applied (so far only this one).

comment:5 in reply to:  3 Changed 14 years ago by Steffen Hoffmann

Keywords: user register verifiy added
Owner: changed from Matt Good to Steffen Hoffmann
Summary: Can't get verification mail[patch] Users can't get verification email

Replying to larsivi:

I have attached a patch that fixes this issue (or at least it fixes how I experienced it). What happens is that the notification code assumes that the username is an email address, even if an email explicitly has been set (in fact nothing is sent if an email isn't explicitly set).

Thanks for sharing your experience on the account verification feature.

I don't know, if your assertion on the AccountManagerPlugin code is true, since it worked flawlessly for me by now. However there is not only TracNotification but AnnouncerPlugin to be taken into account, and I've been always using the latter. I'll have to fix the high priority issues first, but I'll certainly come back at you soon.

This does mean changes to the API, but totally sane ones IMO.

Are there any relevant unittests in place. If so, did you already check them with your changes applied? If not, would you dare to create/propose some. Patch welcome, even building blocks would be certainly appreciated.

comment:6 Changed 13 years ago by Steffen Hoffmann

I'm aware of this pending issue. But I'll have to postpone this to finally get the new release out of the door. But I'll try to resolve this within the next release cycle.

comment:7 Changed 13 years ago by Shane Hill

Hi, just reporting that I'm getting this exact problem. Adding a new user does not seem to send a verification e-mail. Getting that new user to change or update their e-mail address does send a verification e-mail. I'm also using TracNotification. I tried AnnouncePlugin but could not get that to work at all. I have now idea what the AnnouncePlugin was doing, but it would not talk over SendMail given the documentation says it can.

An interesting thing I have found is that as TRAC_ADMIN the new user appears in the user list with full name and e-mail address details. However, if you login as that new user the full name and e-mail details under the "General" preferences tab are blank. Go figure!

comment:8 Changed 13 years ago by Shane Hill

P.S. I'm using Trac 0.12, TracAccountManager 0.4dev and tried using TracAnnouncer-0.12.1dev

comment:9 in reply to:  7 ; Changed 13 years ago by Steffen Hoffmann

Replying to sdhengsoft:

Hi, just reporting that I'm getting this exact problem. Adding a new user does not seem to send a verification e-mail. Getting that new user to change or update their e-mail address does send a verification e-mail.

Thanks for the confirmation. Yes, this is still unresolved, sorry.

An interesting thing I have found is that as TRAC_ADMIN the new user appears in the user list with full name and e-mail address details. However, if you login as that new user the full name and e-mail details under the "General" preferences tab are blank. Go figure!

This is also a known issue, so you may want to follow #9252 for more details and progress.

comment:10 Changed 13 years ago by Shane Hill

Thanks for the update. I'll follow #9252 as well.

comment:11 Changed 13 years ago by Shane Hill

Hey I got TracAnnounce working finally. Still no joy with verification email with newly registered users. The only way seem to get the user to update their email address manually. Then a verification e-mail is sent.

comment:12 Changed 12 years ago by Steffen Hoffmann

Keywords: needinfo added

Please retry with recent trunk code for both plugins, because it has been fixed in TracAnnouncer aka AnnouncerPlugin according to my recent tests. Feedback welcome.

comment:13 in reply to:  description Changed 12 years ago by Steffen Hoffmann

Priority: normallow

Replying to rue@ms5.hinet.net:

I've installed the plugins and enable the all function in this plugin.Once I create new account , users can't get the verification mails. Have you met this situation?

No, I can't reproduce this issue with acct_mgr-0.4 and a recent trunk revision of TracAnnouncer. TracNotification should work as well.

Please re-try to allow for progress, or I'll assume this issue has been resolved by some of the changes before aforementioned plugin release.

comment:14 in reply to:  9 ; Changed 10 years ago by tigerpeng2001@…

Replying to hasienda:

Replying to sdhengsoft:

Hi, just reporting that I'm getting this exact problem. Adding a new user does not seem to send a verification e-mail. Getting that new user to change or update their e-mail address does send a verification e-mail.

Thanks for the confirmation. Yes, this is still unresolved, sorry.

I just ran into same issue (Trac 1.01, account manager 0.4.4) today (Oct. 23, 2014). It has not been solved? Or I miss some setting?

comment:15 in reply to:  14 ; Changed 10 years ago by Steffen Hoffmann

Replying to tigerpeng2001@…:

I just ran into same issue (Trac 1.01, account manager 0.4.4) today (Oct. 23, 2014). It has not been solved? Or I miss some setting?

Clearly comment 12 has been written after a successful test.

You seem to have issues, but your setup is not clear at all. Do you use TracNotification from Trac core? What AcctMgr components are enabled and what account mananger and notification system settings are applied?

comment:16 in reply to:  15 Changed 10 years ago by anonymous

Replying to hasienda:

Replying to tigerpeng2001@…:

I just ran into same issue (Trac 1.01, account manager 0.4.4) today (Oct. 23, 2014). It has not been solved? Or I miss some setting?

Clearly comment 12 has been written after a successful test.

You seem to have issues, but your setup is not clear at all. Do you use TracNotification from Trac core? What AcctMgr components are enabled and what account mananger and notification system settings are applied?

  1. TracNotification is enabled - notifications are CC to account_changes_notify_addresses when create new user (not value for To: field). Password reset does send email to users.
  2. All AcctMgr components, except acct_mgr.register.*, acct_mgr.db.sessionstore, are enabled;
  3. account manager set as below
    [account-manager]
    account_changes_notify_addresses = Ops@mycom.com
    authentication_url =
    db_htdigest_realm = 
    force_passwd_change = true
    hash_method = HtDigestHashMethod
    htdigest_file = 
    htdigest_realm = 
    htpasswd_file = /var/www/svn/repos/conf/svn-auth-conf
    htpasswd_hash_type = crypt
    notify_actions = new,change,delete
    password_file = /var/www/svn/repos/conf/svn-auth-conf
    password_store = HtPasswdStore
    persistent_sessions = true
    refresh_passwd = False
    reset_password = true 
    verify_email = true
    
  1. There may be some clue: when create new user: the 'account_change_notify_addresses' is repeated (show in maillog).
    Oct 24 15:10:38 trac2 sendmail[21813]: s9OFAbad021811: to=<Ops@mycom.com>,<Hcp_Ops@mycom.com>.....
    
Last edited 10 years ago by Ryan J Ollos (previous) (diff)

comment:17 Changed 8 years ago by Ryan J Ollos

Owner: Steffen Hoffmann deleted

comment:18 Changed 5 years ago by Ryan J Ollos

Resolution: worksforme
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The ticket will remain with no owner.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.