Modify

Opened 6 years ago

Closed 2 years ago

#4827 closed enhancement (invalid)

Remove 'login' link from /login page

Reported by: jevans Owned by: coderanger
Priority: normal Component: PermRedirectPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

I don't know why but for some reason we keep getting users that click the login link again after entering their credentials instead of the login button or just pressing enter. Of course all it does is reset the screen.

Attachments (0)

Change History (1)

comment:1 Changed 2 years ago by ejucovy

  • Resolution set to invalid
  • Status changed from new to closed

I think this is a good idea, but it's out of scope for the PermRedirectPlugin. This plugin's only responsibility is catching permission errors and redirecting to the login screen; the implementation of the login screen itself is handled by Trac core and/or AccountManagerPlugin for most environments.

It would be pretty straightforward to implement a filter to remove the "login" link in a separate plugin; it might also be worth filing an upstream request for this change. Either way, closing as "invalid" since PermRedirectPlugin really has no business messing with the contents of the login page.

Add Comment

Modify Ticket

Action
as closed The owner will remain coderanger.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.