datasektionen / cashflow

Django project to manage receipts and reimbursements at Datasektionen
https://cashflow.datasektionen.se
4 stars 13 forks source link

Show pdf:s inline again #144

Closed foodelevator closed 3 months ago

foodelevator commented 1 year ago

(Written by Herman)

Det här är pga en fix på ett säkerhetshål som fanns ganska länge, där man kunde xssa genom att ladda upp pdfer med javascript i. Det fanns två aktuella fixar:

Den senare av dessa blev mergead, men det visade sig att webbläsare inte vill visa pdfer i sandboxade iframes. (researchade lite och hittade ingen bra lösning)

Så frågan är hur vi vill gå vidare. Tar man bort pdf-visningen? Accepterar man att man xss:er finns? (det som kan göras är typ att få folk med attesträtt att attestera mot deras vilja, eller liknande). Eller vill man leta vidare efter någon annan lösning?

foodelevator commented 3 months ago

Fixed in 2fb2150c8d74a19b9207f9d5f2bdc16ad62f0d31

Tried to replace an uploaded pdf file with just a script tag and it did not render anything so maybe unhackable? I just googled how to embed pdf in html and found the object element thingie