Skip to content

Comments

feat: add remove x-frame-options header#2

Merged
lisonge merged 2 commits intolisonge:mainfrom
rocsun:main
Dec 31, 2025
Merged

feat: add remove x-frame-options header#2
lisonge merged 2 commits intolisonge:mainfrom
rocsun:main

Conversation

@rocsun
Copy link
Contributor

@rocsun rocsun commented Dec 31, 2025

x-frame-options与csp头中的frame-ancestors作用相同,但csp头优先级更高,在公司内网存在页面为保证兼容性同时添加了x-frame-options与csp头的frame-ancestors部分,如果只移除csp头反而导致frame页被拦截无法访问

@lisonge lisonge changed the title 移除x-frame-options feat: add remove x-frame-options header Dec 31, 2025
@lisonge lisonge merged commit 5dea6ac into lisonge:main Dec 31, 2025
@lisonge
Copy link
Owner

lisonge commented Dec 31, 2025

image

没注意到,你的代码有语法错误,我重新弄吧

lisonge added a commit that referenced this pull request Dec 31, 2025
@lisonge
Copy link
Owner

lisonge commented Dec 31, 2025

829b597

@rocsun
Copy link
Contributor Author

rocsun commented Dec 31, 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