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

新增短代码reply #1060

Closed
wants to merge 1 commit into from
Closed

Conversation

Deepseaon
Copy link

@Deepseaon Deepseaon commented Jan 20, 2025

新增短代码reply,实现功能隐藏指定内容需回复或登录后可见
添加了对 email 格式的有效性检查(is_email())
使用 esc_sql() 对用户输入的 email 字段进行了转义
使用esc_html()转义(wp_login_url(get_permalink()))避免可能的 XSS 攻击
使用esc_html()转义do_shortcode($content));,避免可能的 XSS 攻击
#1026

@mirai-mamori mirai-mamori changed the base branch from main to canary January 20, 2025 23:14
@mirai-mamori
Copy link
Owner

解决一下冲突 并且请完善一下多语言支持

@Deepseaon
Copy link
Author

解决一下冲突 并且请完善一下多语言支持

好的。请问可以指明一下是哪个部分出现了冲突吗

@mirai-mamori
Copy link
Owner

pr下面就有提示哦This branch has conflicts that must be resolved

@nicocatxzc nicocatxzc mentioned this pull request Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants