alexcrack / alexcrack/angular-ui-notification

Add protection against XSS

Ouverte
#86 1 commentaire 1 réaction 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
529
Forks
164
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

In my app the following code opens a dialog box with "hi" :
`Notification.success("Some message alert('hi')")`

It shows that my app was vulnerable to XSS because Notification was used in that sort of way : `Notification.error("got an error : "+myError);`

And myError is a string that can come from anywhere, and is unsecure.

My fix was to decore Notification service and apply $sanitize on strings.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.