Modify

Opened 14 years ago

Closed 4 years ago

#7576 closed enhancement (fixed)

[patch] Users redirected to login screen with no confirmation may fail to recognize that they are registered

Reported by: anonymous Owned by:
Priority: normal Component: AccountManagerPlugin
Severity: critical Keywords: user register trivial needinfo
Cc: Trac Release: 0.11

Description

When no email verification is used, the user is registered and immediately redirected to login page. Most don't understand this and try registering again and now get message "user already exists".

It should be made clear registered actually succeeded, before redirecting to login.

Attachments (1)

acct_mgr-0.2.1dev_with-7576-fix.png (40.7 KB) - added by Steffen Hoffmann 13 years ago.
Confirmation after successful registration of user «test» in Trac 0.12

Download all attachments as: .zip

Change History (13)

comment:1 Changed 14 years ago by Ryan J Ollos

Summary: users constantly fail to recognice they are registerenUsers are redirected to the login screen with no confirmation email and fail to recognize they are registered

It should be fairly straightforward to use add_notice to display a message to the user and show they are registered.

comment:2 in reply to:  1 Changed 13 years ago by Steffen Hoffmann

Keywords: trivial added
Severity: normalcritical
Summary: Users are redirected to the login screen with no confirmation email and fail to recognize they are registeredUsers redirected to login screen with no confirmation may fail to recognize that they are registered
Type: defectenhancement

Replying to rjollos:

It should be fairly straightforward to use add_notice to display a message to the user and show they are registered.

Right, but a patch is still needed. Only I'm just too tired to do it right now.

comment:3 Changed 13 years ago by Steffen Hoffmann

Keywords: user register added
Summary: Users redirected to login screen with no confirmation may fail to recognize that they are registered[patch] Users redirected to login screen with no confirmation may fail to recognize that they are registered

Good news: While stumbling through the code to debug another issue I spotted a nice place, where this could be hooked into.

I've a nice working solution now, almost trivial, but need to separate from other changes I've prepared in parallel. Patch will be attached later.

Changed 13 years ago by Steffen Hoffmann

Confirmation after successful registration of user «test» in Trac 0.12

comment:4 Changed 13 years ago by Steffen Hoffmann

See screen capture with fix applied to trunk.

This is already an heavily patched test application with i18n support and German translation in place as well. The English original message test would be

Registration has been finished successfully. You may login as user %(user)s now.

comment:5 Changed 13 years ago by Steffen Hoffmann

Resolution: fixed
Status: newclosed

(In [9199]) AccountManagerPlugin: Add message to confirm successful registrations, closes #7576.

comment:6 Changed 11 years ago by Dirk Stöcker

Resolution: fixed
Status: closedreopened

Does not work. See e.g. JOSM Trac which runs 0.4.

comment:7 Changed 11 years ago by Steffen Hoffmann

Indeed, I noticed that too, but only with older Trac version. Nevertheless I see, that JOSM Trac runs on current stable Trac-1.0 (what was not around at the time of the last change to this ticket). Will re-check.

So far I know, that its an issue with Trac 0.11, that has, but doesn't use Chrome.add_notice correctly. There are many more occasions, when Trac-0.12 and later will show a message box, while earlier versions wont.

I refrained from fixing it everywhere on purpose: "works-with-0.12" doesn't break functionality but mildly encourages to upgrade to a more recent Trac version, if someone had a big issue with that.

So you want me to duplicate functionality at least for this occasion? Will do, but mainly because this feature has been requested for 0.11 initially.

comment:8 in reply to:  6 Changed 11 years ago by Steffen Hoffmann

Replying to stoecker:

Does not work. See e.g. JOSM Trac which runs 0.4.

Hm, did tests with Trac 0.11, 1.0 and 1.1dev, but only 0.11 failed to show the success messages.

So do we need to find the issue within JOSM Trac config?

And I'm thinking hard, how to push the success message to the login page for Trac 0.11, because existing workaround relies on rendering messages from the data dict. I fear, that the redirect to /login complicates passing the message a lot, because before the redirect neither the future Request object nor the data dict for templates/login.html are available. Until now only one the (anonymous) user session (something like req.session['chrome_message']) came to my mind.

comment:9 Changed 11 years ago by Steffen Hoffmann

Owner: changed from John Hampton to Steffen Hoffmann
Status: reopenednew

comment:10 Changed 11 years ago by Ryan J Ollos

Keywords: needinfo added

I'd vote for not worrying about this issue on Trac 0.11, unless someone insists loudly and provides a strong case for why they can't upgrade. As mentioned, the concerning issue seems to be that this should work fine with JOSM Trac, so it seems we need more info.

comment:11 Changed 7 years ago by Ryan J Ollos

Owner: Steffen Hoffmann deleted

comment:12 Changed 4 years ago by Ryan J Ollos

Resolution: fixed
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.