Modify

Opened 4 years ago

Closed 4 years ago

#11154 closed defect (fixed)

AttributeError: 'TracHacksHtPasswdStore' object has no attribute 'user_email_verification_requested'

Reported by: Ryan J Ollos Owned by: Steffen Hoffmann
Priority: normal Component: TracHacksPlugin
Severity: normal Keywords:
Cc: Ivanelson Nunes, Chris Nelson Trac Release:

Description

Two users have reported this issue. It appears to be due to the HtPasswdStore implementation in TracHacksPlugin not having been adapted to the AccountManagerPlugin 0.4 API (IAccountChangeListener).

Attachments (0)

Change History (6)

comment:1 Changed 4 years ago by Ryan J Ollos

In 13272:

Refs #11154: Added pass-through implementations of two missing IAccountChangeListener methods (for AccountManagerPlugin 0.4.3).

comment:2 Changed 4 years ago by Ryan J Ollos

[13271] was just a quick workaround until we get some real expertise (hasienda) on the problem.

comment:3 in reply to:  2 ; Changed 4 years ago by Steffen Hoffmann

Replying to rjollos:

[13271] was just a quick workaround until we get some real expertise (hasienda) on the problem.

Approved. Accidentally I've been planning a major rework of the change listener interface in question. But didn't search for existing implementations yet. Once finished, it will no longer blew-up when calling older listeners that naturally will miss methods or have incompatible method signatures.

comment:4 Changed 4 years ago by Chris Nelson

When logging in I see:

An email has been sent to with a token to verify your new email address.

but the e-mail never showed up. Still, I seem to be able to use the system.

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

Replying to ChrisNelson:

When logging in I see:

An email has been sent to with a token to verify your new email address.

but the e-mail never showed up. Still, I seem to be able to use the system.

A bit strange indeed. First, there is no email. It should be mentioned in the text between "has been sent to" and "with a token", but it is not. Second, until providing the valid verification token you should all elevated permissions should be stripped from the authenticated session, with an exception for admin users, but that group should not be challenged in the first place. Btw, any regular user will get challenged again after changing a verified address.

comment:6 in reply to:  3 Changed 4 years ago by Ryan J Ollos

Resolution: fixed
Status: newclosed

Replying to hasienda:

Approved. ...

I'll go ahead and close then, but please reopen if you think there should be additional work done related to this ticket.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Steffen Hoffmann.
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.