Opened 16 months ago

Closed 16 months ago

Last modified 5 months ago

#31359 closed Cleanup/optimization (fixed)

Deprecate using get_random_string without an explicit length

Reported by: Claude Paroz Owned by: Claude Paroz
Component: Utilities Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

django.utils.crypto.get_random_string currently has a default length value (12). I think we should force callers to specify the length value and not count on a default.

Change History (5)

comment:1 Changed 16 months ago by Claude Paroz

Has patch: set

comment:2 Changed 16 months ago by Mariusz Felisiak

Owner: changed from nobody to Claude Paroz
Status: newassigned
Triage Stage: UnreviewedAccepted

comment:3 Changed 16 months ago by Mariusz Felisiak

Triage Stage: AcceptedReady for checkin

comment:4 Changed 16 months ago by Mariusz Felisiak <felisiak.mariusz@…>

Resolution: fixed
Status: assignedclosed

In e663f69:

Fixed #31359 -- Deprecated get_random_string() calls without an explicit length.

comment:5 Changed 5 months ago by Mariusz Felisiak <felisiak.mariusz@…>

In be6e4681:

Refs #31359 -- Made get_random_string()'s length argument required.

Per deprecation timeline.

Note: See TracTickets for help on using tickets.
Back to Top