Epidocs / epita.it

The open source web portal for EPITA websites and projects.
https://epita.it
40 stars 34 forks source link

Remove RTFN from epita.it (XSS vulnerability) #73

Closed utybo closed 3 years ago

utybo commented 3 years ago

RTFN is subject to XSS. Please remove it from the links ASAP. I have no way of reaching out to the creators, please link me a repo somewhere where I could report this properly. I have contacted one of the creators by e-mail, but RTFN does not seem to be actively maintained, so I do not have high hopes that this will get resolved in a timely manner.

Proof

What is even worse is that you have no way to know whether an article is malicious other than checking the page's source code, <script> blocks are invisible.

Although usernames are logged within RTFN's systems, it is very easy through social engineering to get a user's credentials. And even then, exploiting this vulnerability is trivial.

Quick calculations on the CVSS 3.1 calculator indicate a juicy High (8.5/10) severity, yummy!

MartinMarx commented 3 years ago

The issue has been fixed.

All HTML tags are now stripped in news title, content, tags and signature. This fix also applies to previews on the homepage and on the newsgroup details page.

utybo commented 3 years ago

Thanks for the quick action. I'll close both this issue and the PR :)