Skip to content

Composer: Add ezyang/htmlpurifier for ILIAS 12 - #10571

Closed
fhelfer wants to merge 1 commit into
ILIAS-eLearning:trunkfrom
fhelfer:12/dependencies/htmlpurifier
Closed

fhelfer wants to merge 1 commit into
ILIAS-eLearning:trunkfrom
fhelfer:12/dependencies/htmlpurifier

Conversation

@fhelfer

@fhelfer fhelfer commented Nov 28, 2025

Copy link
Copy Markdown
Contributor

This PR suggests adding ezyang/htmlpurifier (v. 4.19) as composer dependency.

General Information:

  • this dependency was already used in ILIAS.
  • License: LGPL-2.1

Usage:

  • components/ILIAS/Html

Wrapped By:

  • \ilHtmlPurifierAbstractLibWrapper (implements \ilHtmlPurifierInterface)

Reasoning:

  • HTMLPurifier is a standards-compliant HTML filter library written in PHP. It will not only remove all malicious code (better known as XSS) with a thoroughly audited, secure yet permissive whitelist, it will also make sure your documents are standards compliant, something only achievable with a comprehensive knowledge of W3C's specifications.
  • We should not try to/do not want to reinvent the wheel in ILIAS and rely on this security library when filtering HTML code.

Maintenance:

Alternatives:

Links:

@fhelfer fhelfer added dependencies Pull requests that update a dependency file php Pull requests that update Php code and removed php Pull requests that update Php code labels Nov 28, 2025
@fhelfer fhelfer changed the title Add ezyang/htmlpurifier for ILIAS 12 Composer: Add ezyang/htmlpurifier for ILIAS 12 Nov 28, 2025
@fhelfer fhelfer changed the title Composer: Add ezyang/htmlpurifier for ILIAS 12 Composer: Add ezyang/htmlpurifier for ILIAS 12 Nov 28, 2025
@matthiaskunkel

Copy link
Copy Markdown
Member

Jour Fixe, 08 DEC 2025: Dependency is accepted for ILIAS 12.

@lscharmer

Copy link
Copy Markdown
Contributor

Merged with PR: #11255

@lscharmer lscharmer closed this Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants