Skip to content

Conversation

@john-traas
Copy link
Contributor

@john-traas john-traas commented Nov 14, 2024

As per this post here we should update to prosemirror-model 1.22.1 as it contains a fix for a vulnerability to xss attacks in the DOMSerialiser.

@github-actions
Copy link

Documentation has been published to https://lundalogik.github.io/lime-elements/versions/PR-3300/

@john-traas john-traas self-assigned this Nov 14, 2024
adrianschmidt
adrianschmidt previously approved these changes Nov 14, 2024
Copy link
Contributor

@adrianschmidt adrianschmidt left a comment

Choose a reason for hiding this comment

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

Seems reasonable.

I just did a very quick smoke test, making sure it was possible to write stuff in the basic example, and that we got the updated value output.

Copy link
Contributor

@adrianschmidt adrianschmidt left a comment

Choose a reason for hiding this comment

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

This should be a fix, so it creates a new release, and it's good if the commit message says what the benefit to the consumer is. Something like:

fix(text-editor): update prosemirror-model to 1.22.1 to fix XSS vulnerability

@john-traas john-traas force-pushed the bump-prosemirror-model-version branch from 7f57618 to a1cccc2 Compare November 14, 2024 12:50
@john-traas
Copy link
Contributor Author

This should be a fix, so it creates a new release, and it's good if the commit message says what the benefit to the consumer is. Something like:

fix(text-editor): update prosemirror-model to 1.22.1 to fix XSS vulnerability

⚡ Updated

@adrianschmidt adrianschmidt enabled auto-merge (rebase) November 14, 2024 13:03
@adrianschmidt adrianschmidt merged commit 55a916c into main Nov 14, 2024
9 checks passed
@adrianschmidt adrianschmidt deleted the bump-prosemirror-model-version branch November 14, 2024 13:07
@lime-opensource
Copy link
Collaborator

🎉 This PR is included in version 37.66.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants