Skip to content
This repository has been archived by the owner on Jan 21, 2024. It is now read-only.

🐛 Fix: Fix comment origin html tag perform #222

Merged
merged 1 commit into from
Aug 4, 2020
Merged

🐛 Fix: Fix comment origin html tag perform #222

merged 1 commit into from
Aug 4, 2020

Conversation

xzhuz
Copy link
Contributor

@xzhuz xzhuz commented Aug 4, 2020

Fix comment origin html tag perform

@JohnNiang JohnNiang requested a review from ruibaby August 4, 2020 03:48
@JohnNiang JohnNiang assigned JohnNiang, ruibaby and xzhuz and unassigned JohnNiang and ruibaby Aug 4, 2020
@JohnNiang JohnNiang added the kind/feature Categorizes issue or PR as related to a new feature. label Aug 4, 2020
@ruibaby
Copy link
Member

ruibaby commented Aug 4, 2020

这样做的目的是？

@xzhuz
Copy link
Contributor Author

xzhuz commented Aug 4, 2020

这样做的目的是？

如果评论的是使用的原始HTML标签， 比如说： <p></p> ，而后台存的确实 &lt;p&gt; , 这样后台使用marked(content)根本解析不了

@ruibaby
Copy link
Member

ruibaby commented Aug 4, 2020

这样做的目的是？

如果评论的是使用的原始HTML标签， 比如说： <p></p> ，而后台存的确实 &lt;p&gt; , 这样后台使用marked(content)根本解析不了

那有没有考虑 script 标签呢？ 本身转义的目的就是防止 xss。

@xzhuz
Copy link
Contributor Author

xzhuz commented Aug 4, 2020

这样做的目的是？

如果评论的是使用的原始HTML标签， 比如说： <p></p> ，而后台存的确实 &lt;p&gt; , 这样后台使用marked(content)根本解析不了

那有没有考虑 script 标签呢？ 本身转义的目的就是防止 xss。

这里不需要防止XSS 攻击吧？这里只是作为文本展示，并不需要有请求。

@ruibaby
Copy link
Member

ruibaby commented Aug 4, 2020

这样做的目的是？

如果评论的是使用的原始HTML标签， 比如说： <p></p> ，而后台存的确实 &lt;p&gt; , 这样后台使用marked(content)根本解析不了

那有没有考虑 script 标签呢？ 本身转义的目的就是防止 xss。

这里不需要防止XSS 攻击吧？这里只是作为文本展示，并不需要有请求。

通过 https://deploy-preview-222--adoring-jang-01641c.netlify.app/ 测试了，没有问题。

@ruibaby ruibaby merged commit 3ceec99 into halo-dev:master Aug 4, 2020
@xzhuz
Copy link
Contributor Author

xzhuz commented Aug 5, 2020

这样做的目的是？

如果评论的是使用的原始HTML标签， 比如说： <p></p> ，而后台存的确实 &lt;p&gt; , 这样后台使用marked(content)根本解析不了

那有没有考虑 script 标签呢？ 本身转义的目的就是防止 xss。

这里不需要防止XSS 攻击吧？这里只是作为文本展示，并不需要有请求。

通过 https://deploy-preview-222--adoring-jang-01641c.netlify.app/ 测试了，没有问题。

可以，现在越来越专业了

JohnNiang pushed a commit to JohnNiang/halo that referenced this pull request Mar 2, 2023
@xzhuz
🐛 Fix: Fix comment origin html tag perform (halo-dev/console#222)
b422242
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ruibaby ruibaby ruibaby approved these changes

Assignees

@xzhuz xzhuz

Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xzhuz @ruibaby @JohnNiang