Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitac22fe4

Browse files
authored
chore: revert network isolation UI (#152)
We're always going to be enabling the soft isolation mode now.
1 parent814b412 commitac22fe4

File tree

7 files changed

+8
-47
lines changed

7 files changed

+8
-47
lines changed

‎App/App.xaml.cs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,6 @@ public App()
9292

9393
services.AddSingleton<IDispatcherQueueManager>(_=>this);
9494
services.AddSingleton<IDefaultNotificationHandler>(_=>this);
95-
services.AddSingleton<ISettingsManager<CoderConnectSettings>,SettingsManager<CoderConnectSettings>>();
9695
services.AddSingleton<ICredentialBackend>(_=>
9796
newWindowsCredentialBackend(WindowsCredentialBackend.CoderCredentialsTargetName));
9897
services.AddSingleton<ICredentialManager,CredentialManager>();
@@ -121,6 +120,7 @@ public App()
121120
// FileSyncListMainPage is created by FileSyncListWindow.
122121
services.AddTransient<FileSyncListWindow>();
123122

123+
services.AddSingleton<ISettingsManager<CoderConnectSettings>,SettingsManager<CoderConnectSettings>>();
124124
services.AddSingleton<IStartupManager,StartupManager>();
125125
// SettingsWindow views and view models
126126
services.AddTransient<SettingsViewModel>();

‎App/Models/Settings.cs‎

Lines changed: 2 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,6 @@ public class CoderConnectSettings : ISettings<CoderConnectSettings>
3434
/// </summary>
3535
publicboolConnectOnLaunch{get;set;}
3636

37-
/// <summary>
38-
/// When this is true Coder Connect will not attempt to protect against Tailscale loopback issues.
39-
/// </summary>
40-
publicboolEnableCorporateVpnSupport{get;set;}
41-
4237
/// <summary>
4338
/// CoderConnect current settings version. Increment this when the settings schema changes.
4439
/// In future iterations we will be able to handle migrations when the user has
@@ -51,21 +46,17 @@ public CoderConnectSettings()
5146
Version=VERSION;
5247

5348
ConnectOnLaunch=false;
54-
55-
EnableCorporateVpnSupport=false;
5649
}
5750

58-
publicCoderConnectSettings(int?version,boolconnectOnLaunch,boolenableCorporateVpnSupport)
51+
publicCoderConnectSettings(int?version,boolconnectOnLaunch)
5952
{
6053
Version=version??VERSION;
6154

6255
ConnectOnLaunch=connectOnLaunch;
63-
64-
EnableCorporateVpnSupport=enableCorporateVpnSupport;
6556
}
6657

6758
publicCoderConnectSettingsClone()
6859
{
69-
returnnewCoderConnectSettings(Version,ConnectOnLaunch,EnableCorporateVpnSupport);
60+
returnnewCoderConnectSettings(Version,ConnectOnLaunch);
7061
}
7162
}

‎App/Services/RpcController.cs‎

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,18 +69,16 @@ public interface IRpcController : IAsyncDisposable
6969
publicclassRpcController:IRpcController
7070
{
7171
privatereadonlyICredentialManager_credentialManager;
72-
privatereadonlyISettingsManager<CoderConnectSettings>_settingsManager;
7372

7473
privatereadonlyRaiiSemaphoreSlim_operationLock=new(1,1);
7574
privateSpeaker<ClientMessage,ServiceMessage>?_speaker;
7675

7776
privatereadonlyRaiiSemaphoreSlim_stateLock=new(1,1);
7877
privatereadonlyRpcModel_state=new();
7978

80-
publicRpcController(ICredentialManagercredentialManager,ISettingsManager<CoderConnectSettings>settingsManager)
79+
publicRpcController(ICredentialManagercredentialManager)
8180
{
8281
_credentialManager=credentialManager;
83-
_settingsManager=settingsManager;
8482
}
8583

8684
publiceventEventHandler<RpcModel>?StateChanged;
@@ -158,7 +156,6 @@ public async Task StartVpn(CancellationToken ct = default)
158156
usingvar_=awaitAcquireOperationLockNowAsync();
159157
AssertRpcConnected();
160158

161-
varcoderConnectSettings=await_settingsManager.Read(ct);
162159
varcredentials=_credentialManager.GetCachedCredentials();
163160
if(credentials.State!=CredentialState.Valid)
164161
thrownewRpcOperationException(
@@ -178,7 +175,6 @@ public async Task StartVpn(CancellationToken ct = default)
178175
{
179176
CoderUrl=credentials.CoderUrl?.ToString(),
180177
ApiToken=credentials.ApiToken,
181-
TunnelUseSoftNetIsolation=coderConnectSettings.EnableCorporateVpnSupport,
182178
},
183179
},ct);
184180
}

‎App/ViewModels/SettingsViewModel.cs‎

Lines changed: 1 addition & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,6 @@ public partial class SettingsViewModel : ObservableObject
1313
[ObservableProperty]
1414
publicpartialboolConnectOnLaunch{get;set;}
1515

16-
[ObservableProperty]
17-
publicpartialboolDisableTailscaleLoopProtection{get;set;}
18-
1916
[ObservableProperty]
2017
publicpartialboolStartOnLoginDisabled{get;set;}
2118

@@ -34,7 +31,6 @@ public SettingsViewModel(ILogger<SettingsViewModel> logger, ISettingsManager<Cod
3431
_connectSettings=settingsManager.Read().GetAwaiter().GetResult();
3532
StartOnLogin=startupManager.IsEnabled();
3633
ConnectOnLaunch=_connectSettings.ConnectOnLaunch;
37-
DisableTailscaleLoopProtection=_connectSettings.EnableCorporateVpnSupport;
3834

3935
// Various policies can disable the "Start on login" option.
4036
// We disable the option in the UI if the policy is set.
@@ -47,21 +43,6 @@ public SettingsViewModel(ILogger<SettingsViewModel> logger, ISettingsManager<Cod
4743
}
4844
}
4945

50-
partialvoidOnDisableTailscaleLoopProtectionChanged(boololdValue,boolnewValue)
51-
{
52-
if(oldValue==newValue)
53-
return;
54-
try
55-
{
56-
_connectSettings.EnableCorporateVpnSupport=DisableTailscaleLoopProtection;
57-
_connectSettingsManager.Write(_connectSettings);
58-
}
59-
catch(Exceptionex)
60-
{
61-
_logger.LogError($"Error saving Coder Connect{nameof(DisableTailscaleLoopProtection)} settings:{ex.Message}");
62-
}
63-
}
64-
6546
partialvoidOnConnectOnLaunchChanged(boololdValue,boolnewValue)
6647
{
6748
if(oldValue==newValue)
@@ -73,7 +54,7 @@ partial void OnConnectOnLaunchChanged(bool oldValue, bool newValue)
7354
}
7455
catch(Exceptionex)
7556
{
76-
_logger.LogError($"Error saving Coder Connect{nameof(ConnectOnLaunch)}settings:{ex.Message}");
57+
_logger.LogError($"Error saving Coder Connect settings:{ex.Message}");
7758
}
7859
}
7960

‎App/Views/Pages/SettingsMainPage.xaml‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -44,12 +44,6 @@
4444
>
4545
<ToggleSwitchIsOn="{x:Bind ViewModel.ConnectOnLaunch, Mode=TwoWay}" />
4646
</controls:SettingsCard>
47-
<controls:SettingsCardDescription="This setting loosens some VPN loop protection checks in Coder Connect, allowing traffic to flow to a Coder deployment behind a corporate VPN. We only recommend enabling this option if Coder Connect doesn't work with your Coder deployment behind a corporate VPN."
48-
Header="Enable support for corporate VPNs"
49-
HeaderIcon="{ui:FontIcon Glyph=&#xE705;}"
50-
>
51-
<ToggleSwitchIsOn="{x:Bind ViewModel.DisableTailscaleLoopProtection, Mode=TwoWay}" />
52-
</controls:SettingsCard>
5347
</StackPanel>
5448
</Grid>
5549
</ScrollViewer>

‎App/Views/SettingsWindow.xaml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99
xmlns:winuiex="using:WinUIEx"
1010
mc:Ignorable="d"
1111
Title="Coder Settings"
12-
Width="600"Height="500"
13-
MinWidth="600"MinHeight="500">
12+
Width="600"Height="350"
13+
MinWidth="600"MinHeight="350">
1414

1515
<Window.SystemBackdrop>
1616
<MicaBackdrop/>

‎Vpn.Proto/vpn.proto‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ message ServiceMessage {
6262
StartResponsestart=2;
6363
StopResponsestop=3;
6464
Statusstatus=4;// either in reply to a StatusRequest or broadcasted
65-
StartProgressstart_progress=5;// broadcasted during startup (used exclusively by Windows)
65+
StartProgressstart_progress=5;// broadcasted during startup
6666
}
6767
}
6868

@@ -214,7 +214,6 @@ message NetworkSettingsResponse {
214214
// StartResponse.
215215
messageStartRequest {
216216
int32tunnel_file_descriptor=1;
217-
booltunnel_use_soft_net_isolation=8;
218217
stringcoder_url=2;
219218
stringapi_token=3;
220219
// Additional HTTP headers added to all requests

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp