Revert "Filter out [ct:xxx] tags to prevent PE client crashes"#3276
Revert "Filter out [ct:xxx] tags to prevent PE client crashes"#3276ACaiCat wants to merge 2 commits intogeneral-develfrom
Conversation
Greptile SummaryThis PR reverts #3193 by removing the Confidence Score: 5/5Safe to merge assuming the target Terraria server version requires PE clients to be on v1.4.5.6 or later. The revert is mechanically clean — the regex, config field, and unused import are all removed consistently with no dangling references. The only non-blocking consideration is whether older PE clients (pre-1.4.5.6) can still connect and be crashed by [ct:xxx] tags, but that is a deployment/versioning question rather than a code defect. No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant PC as Terraria Client (PC/PE)
participant TS as TShock OnChat Handler
participant Srv as Server/Other Players
PC->>TS: Send chat message (may contain [ct:xxx] tags)
Note over TS: Truncate if necessary
TS->>Srv: Forward chat text as-is
Srv-->>PC: Broadcast to all players
Note over PC: PE v1.4.5.6+ handles [ct:xxx] tags without crashing
Reviews (1): Last reviewed commit: "Revert "Filter out [ct:xxx] tags to prev..." | Re-trigger Greptile |
lost-werewolf
left a comment
There was a problem hiding this comment.
Mobile v1.4.5.6 patches all invalid ControlsTouch tags from my testing. Hopefully for good this time.
Reverts #3193
Terraria PE v1.4.5.6 totally resolves the CT tag crash bug