[PR #19606] Enable DECRQCRA at compile time for all Terminal builds #31891

Closed
opened 2026-01-31 09:50:12 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/19606

State: closed
Merged: Yes


Support for DECRQCRA Request Checksum of Rectangular Area was added in #14989, but left disabled at build time because it could be considered a security risk.

In #17895, we unconditionally added a toggle for it to Terminal's settings UI and settings schema (compatibility.allowDECRQCRA). For users on Stable and Preview, it didn't actually enable anything. Whoops.

Since we have a way to turn it off (and in so doing, mitigate the risk) in Terminal, it's high time for us to remove the feature gating.

Conhost doesn't support turning it off for now and so conhost can still have it compiled out, as a treat.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/19606 **State:** closed **Merged:** Yes --- Support for DECRQCRA Request Checksum of Rectangular Area was added in #14989, but left disabled at build time because it could be considered a security risk. In #17895, we unconditionally added a toggle for it to Terminal's settings UI and settings schema (`compatibility.allowDECRQCRA`). For users on Stable and Preview, it didn't actually enable anything. Whoops. Since we have a way to turn it off (and in so doing, mitigate the risk) in Terminal, it's high time for us to remove the feature gating. Conhost doesn't support turning it off for now and so conhost can still have it compiled out, as a treat.
claunia added the pull-request label 2026-01-31 09:50:12 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#31891