Skip to content

fix(files): Apply HTML escaping to all user-controlled input before сoncatenating it into HTML#1768

Merged
max-nextcloud merged 1 commit intomainfrom
fix/Stored-HTML-Injection-in-Nextcloud-Notes-Markdown-Preview-via-Unescaped-Image-Alt-Text
Mar 13, 2026
Merged

fix(files): Apply HTML escaping to all user-controlled input before сoncatenating it into HTML#1768
max-nextcloud merged 1 commit intomainfrom
fix/Stored-HTML-Injection-in-Nextcloud-Notes-Markdown-Preview-via-Unescaped-Image-Alt-Text

Conversation

@JuliaKirschenheuter
Copy link
Copy Markdown
Contributor

@JuliaKirschenheuter JuliaKirschenheuter commented Mar 4, 2026

Fix for: Stored HTML Injection in Nextcloud Notes Markdown Preview via Unescaped Image Alt Text

To be done:

  • All user-controlled content (including alt text) must be properly escaped according to its HTML context before being concatenated into HTML.
  • Any HTML inserted via v-html should be sanitized using an allowlist-based sanitizer.

Done:

Apply HTML escaping to all user-controlled input before concatenating it into HTML

@JuliaKirschenheuter JuliaKirschenheuter self-assigned this Mar 4, 2026
@JuliaKirschenheuter JuliaKirschenheuter marked this pull request as draft March 4, 2026 08:53
@JuliaKirschenheuter JuliaKirschenheuter force-pushed the fix/Stored-HTML-Injection-in-Nextcloud-Notes-Markdown-Preview-via-Unescaped-Image-Alt-Text branch from eb1126b to d9b580a Compare March 4, 2026 17:01
@JuliaKirschenheuter JuliaKirschenheuter marked this pull request as ready for review March 4, 2026 17:01
@JuliaKirschenheuter JuliaKirschenheuter force-pushed the fix/Stored-HTML-Injection-in-Nextcloud-Notes-Markdown-Preview-via-Unescaped-Image-Alt-Text branch 2 times, most recently from fd6c343 to 7597609 Compare March 4, 2026 17:55
…oncatenating it into HTML

Signed-off-by: julia.kirschenheuter <[email protected]>
@max-nextcloud max-nextcloud force-pushed the fix/Stored-HTML-Injection-in-Nextcloud-Notes-Markdown-Preview-via-Unescaped-Image-Alt-Text branch from 7597609 to d90875b Compare March 13, 2026 10:28
@max-nextcloud max-nextcloud merged commit 31134ad into main Mar 13, 2026
34 checks passed
@max-nextcloud max-nextcloud deleted the fix/Stored-HTML-Injection-in-Nextcloud-Notes-Markdown-Preview-via-Unescaped-Image-Alt-Text branch March 13, 2026 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants