mirror of
https://github.com/microsoft/terminal.git
synced 2025-12-10 18:43:54 -06:00
Enable DECRQCRA at compile time for all Terminal builds (#19606)
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. (cherry picked from commit 3e29d2a316c5f2380ffc2b97939e58c9859fadc8) Service-Card-Id: PVTI_lADOAF3p4s4BBcTlzgiA9JY Service-Version: 1.24
This commit is contained in:
parent
08a981d7b7
commit
47da5a11b2
@ -133,10 +133,14 @@
|
||||
<name>Feature_VtChecksumReport</name>
|
||||
<description>Enables the DECRQCRA checksum report, which can be used to read the screen contents</description>
|
||||
<id>14974</id>
|
||||
<stage>AlwaysDisabled</stage>
|
||||
<alwaysEnabledBrandingTokens>
|
||||
<brandingToken>Dev</brandingToken>
|
||||
</alwaysEnabledBrandingTokens>
|
||||
<stage>AlwaysEnabled</stage>
|
||||
<alwaysDisabledBrandingTokens>
|
||||
<!--
|
||||
Since we have a way to turn this on and off in Terminal, we don't need to hide it behind a feature
|
||||
flag any longer; however, conhost has no way to turn it off so let's just compile it out for now.
|
||||
-->
|
||||
<brandingToken>WindowsInbox</brandingToken>
|
||||
</alwaysDisabledBrandingTokens>
|
||||
</feature>
|
||||
|
||||
<feature>
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user