相关于 Thumbnail generation & markdown rendering issue
当 Discourse 下载图像并将其 HTML 替换为 Markdown 语法时——在我们的场景中,结果如下:
[...]<a href="<link_here>" target="_blank"></a>[...]
Discourse 无法渲染该图像。
图像

要解决此问题,您至少需要在上方添加一个空行:
[...]<a href="<link_here>" target="_blank">
</a>[...]
图像
是否有可能允许渲染被 HTML 包围的 Markdown 图像?
@david
5 个赞
david
(David Taylor)
3
让我们在 Meta 上试试这个。我正在一个链接内热链接这张图片:
<a href="https://discourse.org"><img src="..."/></a>

让我们看看当图片被加载时会发生什么…
3 个赞
david
(David Taylor)
4
@Arkshine 看起来效果不错——它已成功渲染了 <a href="https://discourse.org"></a>。
你能分享一下链接周围的更多 HTML 结构吗?
2 个赞
simon
6
3 个赞
david
(David Taylor)
7
啊哈!@simon,升级 WordPress 插件能修复旧帖子吗?还是只修复新帖子?
3 个赞
simon
8
该修复仅适用于通过 WordPress 区块编辑器发布的帖子。如果在 WordPress 上点击“更新 Discourse 主题”按钮,也可以修复旧帖子。除非有人编写脚本遍历旧帖子,否则需要为每篇帖子手动执行此操作。
4 个赞
david
(David Taylor)
9
让我们试试这个 HTML:
<p><img src="..."/></p>

4 个赞
david
(David Taylor)
10
我认为这也应该在 Discourse 中修复,以防万一帖子是由非 WordPress 集成发布的。总结如下:
-
我发布了 <p><img src="..."/></p>。这是完全有效的 HTML。
-
pull_hotlinked_images 获取了图片,并将标记替换为 <p></p>。
-
这无法渲染。
因此,这里有两个可能的修复方案。要么:
或者
- 修复 Markdown 引擎,使其能够渲染
<p></p>。请注意,这在 CommonMark 演示中也无法渲染。
@sam 你知道为什么 Markdown 图片无法在与 <p> 同一行渲染吗?是否有故意这样设计的原因?
3 个赞
sam
(Sam Saffron)
11
这是 CommonMark 规范的一部分
**test**
<p>**test**</p>
我们绝对不想在这里偏离规范。我想我们可以通过为这种情况注入两个换行符来修复拉取热链图片的问题,使其能够兼容。不过我认为这种情况相当罕见,而且某种程度上是自找的。
5 个赞
我认为这并不那么罕见,尤其是当 Discourse 与 WP-Discourse 等流行工具,或任何使用 API 的工具关联时。
请考虑添加一个空行。这似乎不会造成破坏性变更,而且实现起来相当简单。
3 个赞
david
(David Taylor)
13
@Arkshine 我们内部对此进行了大量讨论。对我们来说,关键在于保持内容的完整性,因此换行解决方案可能不会实施。
但我们肯定会采取一些措施——让 pull_hotlinked_images 任务销毁图像是不可接受的。希望很快能拿出解决方案:eyes:
5 个赞
simon
16
解决此问题的一种变通方法是阻止 Discourse 下载远程图片。可以通过将图片域名添加到“禁用图片下载域名”站点设置来实现。此外,也可以通过禁用“将远程图片下载到本地”站点设置,阻止 Discourse 下载所有远程图片。详情请参见 https://meta.discourse.org/t/fix-broken-images-for-posts-created-by-the-wp-discourse-and-rss-plugins/160773。
4 个赞
在我们的案例中,我们无法这样做,因为我们使用的是官方的话题缩略图组件,该组件需要本地图片。我们通过在使用 WP-Discourse 创建话题之前,在内容中任何 <img> 标签前添加换行符来解决这个问题。这不是适用于所有人的解决方案,但对我们有效。Discourse 不支持这种合法用法,有点令人遗憾。
但确实,如果你不受插件/组件的限制,和/或无法在话题创建之前修复内容,这绝对是一个合理的变通方法。
2 个赞
david
(David Taylor)
18
我们仍在计划修复该问题。不幸的是,这是我们的 Markdown 渲染系统中一个较深层次的问题,修复起来比较复杂。但我们一定会解决它——很抱歉耗时这么久!
5 个赞
simon
22
我顺便补充一下,该问题同样影响通过 Discourse RSS 插件创建的包含图片的帖子。
6 个赞
simon
23
抱歉在这个话题中多次发帖,但该问题同样影响通过我们的 Zendesk 插件创建的帖子中的图片,前提是启用了“从 Zendesk 同步评论”设置。这种情况的难点在于无法预先知道图片的来源,因此将图片 src 添加到“禁用图片下载域名”设置中的变通方法将不起作用。
是否有一种方法可以在图片标签被 HTML 标签包裹时,阻止远程图片下载到本地?
1 个赞
sam
(Sam Saffron)
24
恐怕这完全不可行。如果我们这样做,第三方就可以通过在论坛中注入追踪 GIF 来跟踪使用情况。下载远程图片实际上是一项安全功能。
相反,我认为我们需要一个更“智能”的系统,其工作方式类似于 @tgxworld 几年前构建的图片重映射器:从 HTML 反向工作,并通过重新渲染来确保变更的稳定性。很遗憾,这是一个非常复杂的改动。
4 个赞
angus
(Angus McLeod)
25
这个问题又出现了
只是随口说说,我在想我们是否能绕过这个棘手的问题(即 HTML 到 Markdown 的转换)。再回顾一下(仅为了帮助理清思路):
-
Discourse 支持导入 HTML 以创建帖子内容(例如来自 WP Discourse 的 HTML)。
-
在某些上下文中,用户期望原始 HTML 的完整性被完全保留。
-
此处的“完整性”至少包含两个方面:
- 内容的渲染方式,例如换行符
- 媒体资源的托管位置,例如将图片下载到本地以避免图片链接失效,或出于安全考虑
-
HTML 到 Markdown 的转换可能会影响第一种完整性,但目前为了保障第二种完整性,这种转换是必要的。
因此,针对某些导入的帖子,一种可能的解决方案是直接将导入的 HTML 存储为已渲染的帖子内容,同时让 pull_hotlinked_images 作业支持下载此类内容中的图片,而无需将 img 标签转换为 Markdown。
是的,更简单地说,代码或许可以支持下载热链接图片,而无需将 img 标签转换为 Markdown。对于此类帖子,你将在已渲染的内容中插入下载后的图片 URL,而不是原始 URL(而非原始 HTML 中的 URL)参考此处。
3 个赞
sam
(Sam Saffron)
26
棘手的问题在于,如何编辑带有该标志的帖子。编辑器将处于原始 HTML 模式,整个工具栏也会失效,等等。
1 个赞