Modify

Opened 9 years ago

Closed 7 years ago

#12186 closed defect (wontfix)

Registering new user sending to wrong mail address

Reported by: massimo.b@… Owned by:
Priority: normal Component: AnnouncerPlugin
Severity: normal Keywords: configuration
Cc: Ryan J Ollos Trac Release: 1.0

Description

Using AccountManagerPlugin and AnnouncerPlugin I have an issue when registering new users.

After "Register" and "Login" with...

Username: FBar
Password: ***
Email: foo.bar@localdomain

Trac reports that an email has been sent to foo.bar@localdomain, but in the logs I see that:

02-10 10:47:31,664 Trac[notification] INFO: Sending notification through SMTP at smtp03:25 to [u'FBar@localdomain']

..which of course is the wrong address.

This is the configuration:

[account-manager]
account_changes_notify_addresses = admin@localdomain
allow_delete_account = false
authentication_url = 
db_htdigest_realm = 
force_passwd_change = true
hash_method = HtDigestHashMethod
htdigest_file = 
htdigest_realm = 
htpasswd_file = /mnt/data/trac/projects/trac-test/trac.htpasswd
htpasswd_hash_type = crypt
login_attempt_max_count = 2
notify_actions = new,change,delete
password_file = 
password_store = HtPasswdStore
persistent_sessions = true
refresh_passwd = False
user_lock_max_time = 86400
user_lock_time = 15
user_lock_time_progression = 2
verify_email = true

[announcer]
ctxtnav_names = 
default_email_format = text/plain
email_address_resolvers = SpecifiedEmailResolver, SessionEmailResolver, DefaultDomainEmailResolver
email_enabled = true
email_from = project@localdomain
email_from_name = Trac (project)
email_sender = SmtpEmailSender
email_subject_prefix = __default__
email_to = undisclosed-recipients:
joinable_groups = developers,management
mime_encoding = base64
use_public_cc = false
use_threaded_delivery = false

Attachments (1)

trac.ini (7.7 KB) - added by massimo.b@… 9 years ago.
trac.ini

Download all attachments as: .zip

Change History (8)

comment:1 Changed 9 years ago by Steffen Hoffmann

Cc: Ryan J Ollos added
Component: AccountManagerPluginAnnouncerPlugin
Keywords: configuration added

Thanks for taking care to report your issue.

Still your report

  1. initially ignores the premiere way for local installation and configuration issues, our trac-users MailingList as stated at the /newticket page explicitly (In general it's not a gentle reflex ask for assistance by asserting a software defect when encountering issues, you see?),
  2. completely lacks version information for all three relevant packages (Trac, AccountManager and TracAnnouncer),
  3. doesn't mention specific enabled and disabled modules of these packages,
  4. won't work due to missing required smtp sender configuration section [smtp] (see your TracIni page for hints)
  5. ultimately seems related to TracAnnouncer configuration (the cited INFO log does not originate from AccountManager)

I'm willing to help, even if the mailing-list would be appropriate for sharing info to other users coming after you too, but you would need to provide aforementioned details in a timely manner. And be prepared to get this finally closed as invalid defect claim anyway.

comment:2 Changed 9 years ago by anonymous

Sorry for the incomplete information:

  • trac-1.0.3
  • TracAccountManager-0.5dev_r14363-py2.7.egg
  • TracAnnouncer-1.0dev_r13984-py2.7.egg

Attached you find the complete trac.ini.

Changed 9 years ago by massimo.b@…

Attachment: trac.ini added

trac.ini

comment:3 Changed 9 years ago by massimo.b@…

Right, after disabling TracAnnouncer completely, the correct email address is used.

comment:4 Changed 9 years ago by massimo.b@…

I have to correct. TracAnnouncer is disabled now and the verification mail is still sent to <username>@<default_domain> eventhough an explicit fully qualified email address was given.

So this bug is rather about AccountManager or my configuration.

comment:5 Changed 9 years ago by Massimo

Getting closer, this issue seems to be related to #12228, as deleting those lines from the config results in using the correct email-adress for verification:

account_changes_notify_addresses = mo@localdomain
notify_actions = new,change,delete

comment:6 Changed 7 years ago by Ryan J Ollos

Owner: Steffen Hoffmann deleted

comment:7 Changed 7 years ago by Ryan J Ollos

Resolution: wontfix
Status: newclosed

Please upgrade to Trac 1.2, which has integrated the core of AnnouncerPlugin. Please raise the issue on the trac:MailingList if you encounter the issue with Trac 1.2.

AccountManager will be modified to use the new notification API in #13124.

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.