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

Markdown bug injecting ‘klzzwxh’ into pages, need a fix and tests to ensure it stays fix #1285

Closed
danbri opened this issue Aug 8, 2016 · 7 comments
Assignees
Labels
site tools + python code Infrastructural issues around schema.org site. Most can ignore this!

Comments

@danbri
Copy link
Contributor

danbri commented Aug 8, 2016

@RichardWallis suggested making a "test to try all Term pages on http://schema.org looking for ‘klzzwxh’"

+1

Immediate fixes and more diagnosis also needed, we've repaired health-lifesci.schema.org but the markdown library seems prone to this so unit tests are essential to avoid future recurrence of the problem.

@danbri danbri added the site tools + python code Infrastructural issues around schema.org site. Most can ignore this! label Aug 8, 2016
@AymenLoukil
Copy link

Search page http://schema.org/docs/search_results.html#q=klzzwxh is showing klzzwxh

@danbri danbri changed the title Test needed to check ‘klzzwxh’ doesn't appear in user-facing html Markdown bug injecting ‘klzzwxh’ into pages, need a fix and tests to ensure it stays fix Aug 9, 2016
@danbri
Copy link
Contributor Author

danbri commented Aug 9, 2016

We decided to switch to a different markdown library, plus a custom function for parsing wikilink-style link syntax.

@danbri
Copy link
Contributor Author

danbri commented Aug 10, 2016

Deployed! Cleanup tasks -

  • expunge old library (and file a bug for them, although we can't reproduce it reliably)
  • updated scripts/*py to use markdown2
  • given the library switch do we still need to integrate tests for klzzwxh or is that string tied to the oddities of the first library? (if in doubt, test, I guess...)
  • anything else @RichardWallis ?

@RichardWallis
Copy link
Contributor

This problem was specific to the Markdown library being used.

This has been replaced by another (Markdown2) library, implemented in an entirely different way so 'this' problem will not reappear in this form.

Hense closing.

@danbri
Copy link
Contributor Author

danbri commented Aug 19, 2016

@RichardWallis can you confirm everything in scripts/ uses the new lib?

@RichardWallis
Copy link
Contributor

Yes - no code including scripts and tests make reference to the old Markdown library.

Commit 661088b removes references to it in SOFTWARE_README.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
site tools + python code Infrastructural issues around schema.org site. Most can ignore this!
Projects
None yet
Development

No branches or pull requests

3 participants