fix: Randomize key length to make it harder to brute force
This commit is contained in:
parent
59c7fb9e0d
commit
105b398c68
1 changed files with 2 additions and 1 deletions
|
|
@ -3,11 +3,12 @@
|
|||
|
||||
import frappe
|
||||
from frappe.model.document import Document
|
||||
from random import randrange
|
||||
|
||||
|
||||
class DocumentShareKey(Document):
|
||||
def before_insert(self):
|
||||
self.key = frappe.generate_hash(length=32)
|
||||
self.key = frappe.generate_hash(length=randrange(25, 35))
|
||||
if not self.expires_on and not self.flags.no_expiry:
|
||||
self.expires_on = frappe.utils.add_days(
|
||||
None, days=frappe.get_system_settings("document_share_key_expiry") or 90
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue