Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HMTL being included in post Metadata, and not scrubbed when removed from post. #83

Closed
ghost opened this issue Mar 9, 2019 · 3 comments
Assignees
Labels
Milestone

Comments

@ghost
Copy link

ghost commented Mar 9, 2019

Describe the bug

When using html for inserting images, video or even just centering images, the html code is being included in the metadata desriptions. This is viewable both on google search results, and posted links to sites like Mastodon. Futhermore, deleting the HTML code from the post does not remove the html text from the metadata.

Steps to reproduce (if necessary)

Steps to reproduce the behavior:

  1. Publish a post with html code, within first few lines of post.
  2. Click on publish
  3. post link to post on site such as Mastodon. Code will show in description.

Expected behavior

Code should be ignored and not included in metadata descriptions.

Application configuration

https://write.as

  • Federation enabled? yes

Version or last commit:
https://write.as

@thebaer
Copy link
Member

thebaer commented Mar 13, 2019

Hey, thanks for reporting this. Right now we strip out Markdown for the post descriptions, so we'll need to strip out HTML, as well.

As for the metadata not updating after the HTML has been removed, is that happening when you view the same post on sites like Mastodon? Or when you view source on the post itself? If it only happens on sites like Mastodon, that's likely just because it's caching the metadata.

@thebaer thebaer added the bug label Mar 13, 2019
@thebaer thebaer added this to the 1.0 milestone Mar 13, 2019
@ghost
Copy link
Author

ghost commented Mar 13, 2019 via email

@thebaer thebaer self-assigned this Mar 14, 2019
@thebaer
Copy link
Member

thebaer commented Mar 14, 2019

Yeah, sounds like that's Mastodon caching the metadata. I'm not sure how to invalidate it, unfortunately... But I fixed this, so that won't happen anymore. The change should be deployed on Write.as within the next day.

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

No branches or pull requests

1 participant