Opened 7 years ago

Closed 7 years ago

#5870 closed defect (fixed)

ignoring everything that can occur during login request is a bad idea

Reported by: techtonik Owned by: pedro
Priority: normal Component: NoAnonymousPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11


Right now NoAnonymousPlugin is written so that it ignores any exceptions that occur during login phase. This is probably a bad idea.


  • noanonymous/

    3232        except RequestDone:
    3333            # Reraise on redirect
    3434            raise
    35         except Exception:
    36             # It is possible the error we got called on happened inside
    37             # the _get_panels call. Be sure to ignore it.
    38             pass
    40         return template, data, content_type
    41  No newline at end of file
     36        return template, data, content_type

Attachments (0)

Change History (1)

comment:1 Changed 7 years ago by techtonik

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

Fixed in r6611.

Add Comment

Modify Ticket

as closed The owner will remain pedro.
The resolution will be deleted. Next status will be 'reopened'.

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

Note: See TracTickets for help on using tickets.