fix(weixin): MEDIA tag regex now covers .md, .json, .yaml, .yml, .toml, .log#32611
fix(weixin): MEDIA tag regex now covers .md, .json, .yaml, .yml, .toml, .log#32611wali-reheman wants to merge 1 commit into
Conversation
….yml, .toml, .log These common file types were silently leaking as raw text on WeChat because the MEDIA tag regex didn't recognize them. Fixes NousResearch#32601.
|
Duplicate of #29609 which derives the extension allowlist dynamically from SUPPORTED_DOCUMENT_TYPES instead of hardcoding. Part of a saturated cluster of 5+ competing PRs (#22492, #30588, #32398, #32604, #32358). Also see #32601 (the issue this fixes) which was already triaged as part of the same cluster. |
|
Superseded by #34844, which consolidates this cluster. This PR widens the Closing as superseded — thanks for surfacing and helping pin down this bug; it was part of getting the full fix right. See #34844. |
weixin MEDIA tags for .md/.json/.yaml/.yml/.toml/.log were leaking as raw text instead of being extracted as file attachments — the regex whitelist didn't include them. fixes #32601