Release 3.2.5#1003
Merged
Merged
Conversation
…tial overwrite When GET /config/get returns config to the frontend, sensitive fields (password, token, api_key, secret) are masked as '********'. If the user changes any non-sensitive setting (e.g. ssl: true → false) and saves, the frontend sends back the masked placeholder verbatim. The backend was saving it directly, overwriting the real credential with the literal string '********', breaking authentication silently. Add _restore_sensitive() to substitute any '********' placeholder with the current in-memory value before writing to disk. Non-sensitive fields and genuinely new values are unaffected. Fixes #995
The per-bangumi default in Bangumi model already uses \d+-\d+ to filter batch/collection torrents (e.g. '01-13'). The global RSSParser default was inconsistently using \d+-\d (one fewer +), which has subtly different matching behaviour. Align both defaults to \d+-\d+ for consistency.
When an MCP client disconnects, Starlette tries to call the return value of the handle_sse endpoint as an ASGI response. Since the function had no return statement it returned None, causing: TypeError: 'NoneType' object is not callable The mcp library's own SseServerTransport docstring explicitly documents this requirement. Fix by adding 'return Response()' and importing starlette.responses.Response.
Thanks for the fix! The regex alignment makes sense.
Thanks for catching this! Clean fix for the SSE disconnect issue.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
更新内容
********覆盖的问题 ([错误报告]qbtorrent连接失败 #995)\d+-\d为\d+-\d+(fix(config): align default filter regex \d+-\d to \d+-\d+ #999)handle_sse返回 NoneType 错误 (fix(mcp): return Response() from handle_sse to prevent NoneType error on disconnect #1000)测试