Modify ↓

Opened 15 years ago

Closed 15 years ago

#5870 closed defect (fixed)

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

Reported by: anatoly techtonik Owned by: Pedro Paixao
Priority: normal Component: NoAnonymousPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

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

source:noanonymousplugin/0.11/noanonymous/filter.py@4369​

  • noanonymous/filter.py

     
    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
    3935
    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 15 years ago by anatoly techtonik

Resolution: → fixed
Status: new → closed

Fixed in r6611.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Pedro Paixao.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

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

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.