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

[Bug] 🐛 部分 RSS 客户端要求需要 content-type 请求头 #117

Closed
PBK-B opened this issue Jun 13, 2024 · 0 comments · Fixed by #118
Closed

[Bug] 🐛 部分 RSS 客户端要求需要 content-type 请求头 #117

PBK-B opened this issue Jun 13, 2024 · 0 comments · Fixed by #118
Assignees
Labels
bug Something isn't working

Comments

@PBK-B
Copy link
Collaborator

PBK-B commented Jun 13, 2024

Bug 描述
部分 RSS 客户端需要正确的 content-type: application/rss+xml 响应头,如 https://github.com/FreshRSS/FreshRSS

复现步骤

$ curl -I https://bin.zmide.com/?feed=rss2
HTTP/2 200 
server: nginx
date: Thu, 13 Jun 2024 03:57:02 GMT
content-type: application/rss+xml; charset=UTF-8
last-modified: Fri, 07 Jun 2024 13:42:31 GMT
etag: "a1e9a6ed990a9be2dfee53c813b18255"
strict-transport-security: max-age=31536000

$ curl -I https://direct.xeu.life/sub/rss.xml
HTTP/2 404 
date: Thu, 13 Jun 2024 03:57:45 GMT
content-type: text/plain;charset=UTF-8
access-control-allow-origin: *
vary: *
access-control-allow-headers: Authorization, content-type
access-control-allow-methods: *
access-control-expose-headers: *
cf-placement: local-LHR
server: cloudflare
cf-ray: 89xxxxxx1417f-LHR
alt-svc: h3=":443"; ma=86400

期望行为
部分 RSS 客户端需要正确的 content-type: application/rss+xml; charset=UTF-8 响应头

截图
nil

环境变量
nil

@PBK-B PBK-B changed the title [Bug] 部分 RSS 客户端要求需要 content-type 请求头 [Bug] 🐛 部分 RSS 客户端要求需要 content-type 请求头 Jun 13, 2024
@OXeu OXeu assigned PBK-B and unassigned OXeu Jun 13, 2024
@OXeu OXeu mentioned this issue Jun 13, 2024
@OXeu OXeu closed this as completed in #118 Jun 13, 2024
OXeu added a commit that referenced this issue Jun 13, 2024
* feat: update RSSService to include feed.xml and response headers
@OXeu OXeu added the bug Something isn't working label Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants