Handle no topic link safely

[anthonydillon]

Someone added a topic without a link:

This topic should be parsed and ignored if it does not match minimum criteria.

This feature should be tested to ensure to have hardening for malformed tables content.

[nottrobin]

Would fix https://sentry.is.canonical.com/canonical/ubuntu-com/issues/14414/

[carkod]

I don't know if there is a generic way to do this. But the only way I can think of is a pretty herculean task... Because they could make any error anywhere, so will need to check every cell in the table and make sure:

• URLs:
  • Start with https:// or It actually links to an existent discourse post
  • Path starts with /engage/ and contains a string with all lowercase, no numbers, no strange punctuation marks...
  • If the markdown is formatted correctly according to beautiful soup, so any missing brackets and parenthesis need to be checked.
• Type needs to be a selection of items, namely webinar, blog, whitepaper, event...
• Tags contain only text, with commas, no numbers or other punctuation marks.
• Language matches a country code format matching our templates. So it must be a lowercase, no numbers, no punctuation marks string.
• Active column needs to match either the string true or false.
• Number of columns being added (if there is an additional pipe with empty content or not will displace the table)
• Possible removal of headers in the table or existence of the selection of tables that are currently in place.
• All of the above for takeovers too...

And if any of these does not happen return the errored index and finish the loop with an error?
And I am probably missing some more checks here...

[SirSamTumless]

What's happening with this?