alexcrack / alexcrack/angular-ui-notification

Add protection against XSS

Offen
#86 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
JavaScript
Sterne
529
Forks
164
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.