alexcrack / alexcrack/angular-ui-notification

Add protection against XSS

Aperta
#86 1 commento 1 reazione 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
529
Fork
164
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.