fix: Twitter/X thread finisher posting with HTML tags #1057
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Previously, Twitter/X was posting thread finisher (message ending signature) with HTML
elements hardcoded into the message text. This was because the thread_finisher content was being used directly without HTML stripping.
This fix wraps the thread_finisher text with stripHtmlValidation() to remove HTML tags and convert them to plain text, consistent with how other platforms (Bluesky, Threads, Mastodon) handle this feature.
Changes:
The working set up is live and I've used it. Here's the proof of it working fine:
https://x.com/Meysamazing/status/1987512859727729141?s=20
fixes #1013
Checklist:
Put a "X" in the boxes below to indicate you have followed the checklist;