Opened 4 years ago

Last modified 4 years ago

#31267 closed Cleanup/optimization

Missing tests cases for credential validation in URLs — at Version 3

Reported by: Dennis Schwertel Owned by: nobody
Component: Core (Other) Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description (last modified by Dennis Schwertel)

Hi,
I had an issue with a behavior change introduced with commit cdcf4164bec9dc09465424d7042c3f9d4f0f1fdc. The commit changed the validation of credentials in URLs. This change was not fully covered by test cases and my existing objects now failed to validate. Before the change, credentials in the form of http://:password@host.com where valid.

This is not a big issue as the change fixed an incorrect behavior. I would like to add test cases to cover the current new behavior so changes to the validation are known and intentional.

Change History (3)

comment:1 by Dennis Schwertel, 4 years ago

Pull request can be found here. ​https://github.com/django/django/pull/12448

comment:2 by Dennis Schwertel, 4 years ago

CLA signed and send over.

Looking forward to any feedback!

comment:3 by Dennis Schwertel, 4 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.
Back to Top