-
Notifications
You must be signed in to change notification settings - Fork 8.7k
Labels
Area-SettingsIssues related to settings and customizability, for console or terminalIssues related to settings and customizability, for console or terminalIssue-BugIt either shouldn't be doing this or needs an investigation.It either shouldn't be doing this or needs an investigation.Needs-Tag-FixDoesn't match tag requirementsDoesn't match tag requirementsPriority-1A description (P1)A description (P1)Product-TerminalThe new Windows Terminal.The new Windows Terminal.Severity-CrashCrashes are real bad news.Crashes are real bad news.
Milestone
Description
Windows Terminal version
1.22.250314001 Portable (probably other versions as well)
Windows build number
10.0.26100.4061
Other Software
N\A
Steps to reproduce
Open settings UI, make a change click save > WT crashes
Expected Behavior
An error should be raised if settings.json is not writeable, but WT should not crash.
Actual Behavior
WT Crashes. If folder is writeable but not the settings.json file a settings.json.tmp is created
Metadata
Metadata
Assignees
Labels
Area-SettingsIssues related to settings and customizability, for console or terminalIssues related to settings and customizability, for console or terminalIssue-BugIt either shouldn't be doing this or needs an investigation.It either shouldn't be doing this or needs an investigation.Needs-Tag-FixDoesn't match tag requirementsDoesn't match tag requirementsPriority-1A description (P1)A description (P1)Product-TerminalThe new Windows Terminal.The new Windows Terminal.Severity-CrashCrashes are real bad news.Crashes are real bad news.
Type
Projects
Status
Cherry Picked
Status
Cherry Picked
Relationships
Development
Select code repository
Activity
Zeroes1 commentedon May 18, 2025
canary terminal-1.24.1351.0 same.
DHowett commentedon May 28, 2025
Thanks! I could have sworn we had a duplicate for this, but it appears that we don't. Congrats!
Though: why is it not writable? 🙃
PGomersall commentedon May 28, 2025
@DHowett
I had a script to automate updating and or creation of the portable in C:\WT which runs as system. On some systems the default permissions on C:\ and therefore the inherited C:\WT did not give users more than read and execute. I added code in my script to deal with this, but wt should not crash regardless trying to write to the settings file that is effectively locked, hence my bug ticket.
Display a warning if SUI is unable to write to the settings file (mic…
Display a warning if SUI is unable to write to the settings file (#19027
Display a warning if SUI is unable to write to the settings file (#19027