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

bugfix:修复当网页中设置no-referrer时安全域名配置 & 修复安全域名正则表达式不生效的问题 || bugfix: Fix the problem of safe domain name configuration & fix the problem that the safe domain name regular expression does not take effect when no-referrer is set in the web page #2663

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

axiaoxin
Copy link

@axiaoxin axiaoxin commented Aug 12, 2024

  • 修复当网页中设置no-referrer时安全域名配置无法生效的问题。
  • 修复安全域名正则表达式不生效的问题。

  • Fixed the problem that the secure domain name configuration cannot take effect when no-referrer is set in the web page.
  • Fixed the problem that the regular expression of safe domain name does not take effect.

- 修复当网页中设置no-referrer时安全域名配置无法生效的问题。
- 修复安全域名正则表达式不生效的问题。
@github-actions github-actions bot changed the title bugfix:修复当网页中设置no-referrer时安全域名配置 & 修复安全域名正则表达式不生效的问题 bugfix:修复当网页中设置no-referrer时安全域名配置 & 修复安全域名正则表达式不生效的问题 || bugfix: Fix the problem of safe domain name configuration & fix the problem that the safe domain name regular expression does not take effect when no-referrer is set in the web page Aug 12, 2024
@axiaoxin
Copy link
Author

axiaoxin commented Aug 12, 2024

修复问题1: 网页meta中设置no-referrer后,配置安全域名无法生效,因为获取不到referrer,进不了那个if。 #52
改为referrer获取不到是获取请求头中的origin来判断

修复问题2: 安全域名未正确实现正则表达式的判断,domain.test 会报错,因为是字符串不是正则表达式


Fixed problem 1: After setting no-referrer in the web page meta, the secure domain name configuration cannot take effect because the referrer cannot be obtained and the if cannot be entered. #52
If the referrer cannot be obtained, it can be judged by obtaining the origin in the request header.

Fix problem 2: The secure domain name does not correctly implement the regular expression judgment, domain.test will report an error because it is a string and not a regular expression

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.

1 participant