Skip to content

chore: Update ck5 deps to use latest version#1143

Open
carla-at-wiris wants to merge 1 commit intorelease/v8.15.3from
chore/update-deps
Open

chore: Update ck5 deps to use latest version#1143
carla-at-wiris wants to merge 1 commit intorelease/v8.15.3from
chore/update-deps

Conversation

@carla-at-wiris
Copy link
Contributor

Description

Update CKEditor 5 dependencies so that it uses the latest version, which includes a security patch, v.47.6.0.

Type of Change

  • Chore (non-breaking change that doesn't add any functionality)

How should be tested?

  • Go to the staging demo.
  • Open the conole and write CKEDITOR_VERSION
  • Validate that the version is equal or higher than 47.6.0

}
/* Work around a security issue with comments inside attributes */
if (SAFE_FOR_XML && regExpTest(/((--!?|])>)|<\/(style|title|textarea)/i, value)) {
if (SAFE_FOR_XML && regExpTest(/((--!?|])>)|<\/(style|script|title|xmp|textarea|noscript|iframe|noembed|noframes)/i, value)) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

¿Esto viene de gratis?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sip, es autogenerado, asi que viene de los cambios de la nueva version de ck5 que pues hace que este fichero autogenerado cambie así

Copy link
Contributor

@jgonzalez-at-wiris jgonzalez-at-wiris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Aprobado en modo 👁️ . Confio (no me queda otra)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants