Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

html-react-parser XSS issue #1

Open
HugoSave opened this issue Apr 25, 2021 · 1 comment
Open

html-react-parser XSS issue #1

HugoSave opened this issue Apr 25, 2021 · 1 comment

Comments

@HugoSave
Copy link
Contributor

Då somliga användare kommer kunna redigera vissa sidor genom att indirekt skriva html i vår editor så finns det en eventuell risk att någon kan lyckas få in skadlig/irriterande kod. I dagsläget används paketet html-react-parser för att omvandla en sträng med html-kod till react komponenter som vi kan rendera. html-react-parser garanterar däremot inte att man inte kan injecera godtycklig javascriptkod som då skulle köras på alla webläsare som går in på sidan. https://github.com/remarkablemark/html-react-parser/issues/94

I tråden ovan föreslår dem bland annat att man först kan sanitera användarkoden med https://github.com/cure53/DOMPurify vilket tydligen inte är ett alltför stort paket men det ger såklart lite overhead.

Här är ett exempel i vår kodbas där vi använder html-react-parser (via en funktion som heter parse):
https://github.com/Fysiksektionen/Hemsida/blob/main/client/src/components/content_object_renderers/blocks/RichTextCOR.tsx

Jag vet egentligen inte om det är ett problem eller inte men tänkte skapa denna tråd här så kan man lämna åsikter i kommentarerna. Värsta scenario är väl att användarens kakor och liknande data kan skickas vidare till tredjepart. Vilket kanske kan ge andra tillgång till deras konto på fysiksektionen? Men jag har inte riktigt koll på vad man kan och inte kan göra med XSS attacker och till vilken grad html-react-parser är sårbar.

@Ejemyr
Copy link
Member

Ejemyr commented Apr 25, 2021

Jättebra catch!
Hade inte tänkt på det, men absolut något vi måste tänka på 👍

@HugoSave HugoSave transferred this issue from Fysiksektionen/Hemsida May 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants