Changes between Initial Version and Version 1 of Ticket #29206


Ignore:
Timestamp:
Mar 13, 2018, 8:55:56 PM (7 years ago)
Author:
Tim Graham
Comment:

Yes, that shouldn't generate an exception.

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #29206

    • Property Component Uncategorized → contrib.auth
    • Property Triage Stage Unreviewed → Accepted
    • Property Summary Validation Error and 500 when a reset password link contains a bad UIDB64 UUID → ValidationError when a reset password link contains a bad UIDB64 UUID
    • Property Type Uncategorized → Bug

  • Ticket #29206 – Description

    initial v1  
    5858}}}
    5959
    60 The error is raised when **user = UserModel._default_manager.get(pk=uid)** is executd, while I think a ValidationError is a proper exception I'm wondering if that couldn't be catched within that try/except. I feel is a bit weird getting a 500 if somebody tries to forge a URL for example.
     60The error is raised when `user = UserModel._default_manager.get(pk=uid)` is executed, while I think a ValidationError is a proper exception I'm wondering if that couldn't be catched within that try/except. I feel is a bit weird getting a 500 if somebody tries to forge a URL for example.
    6161
Back to Top