hackmdio/codimd

<iframe> tag cause open redirect

Ouverte

#959 ouverte le 18 sept. 2018

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (0 personne assignée)JavaScript (1 038 forks)batch import
Hacktoberfesthelp wantedsecurity

Métriques du dépôt

Stars
 (8 949 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

If the source website has the script like this:

<script type="text/javascript">
if(window != top) {
    top.location.href = location.href;
}
</script>

It may cause a open redirect issue on codimd. I use www.plurk.com which has anti-clickjacking code to demo.

Demo Link in demo.codimd.org

<iframe src="https://www.plurk.com/k1tten_">

Broswer verison:

Safari 11.0.2: triggered
Firefox Quantum 62.0 : triggered
Chrome 68.0.3440.106: not triggered

Guide contributeur

Direction de recherche
Examinez la gestion des iframes dans le rendu des notes et assurez vous que toutes les URL définies via top.location sont filtrées ou bloquées.
Stack technique
javascript
Domaine
security
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitWeb Security
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70