okfde / django-filingcabinet

PDF document viewer Django app
13 stars 4 forks source link

🩹 Validate that query collection id is digit #49

Closed pajowu closed 1 year ago

pajowu commented 1 year ago

Note: This assumes that collection ids are always ints, i.e. if we ever switch to uuids, strings, etc this breaks. Alternative would be to catch ValueError in this line, which imho is also not perfect