Skip to content

Commit 84aed21

Browse files
committed
It`s 2017, time for .NET 4.6
- Fixes #86
1 parent 463b214 commit 84aed21

5 files changed

Lines changed: 46 additions & 32 deletions

File tree

SimpleDnsCrypt/App.config

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
<?xml version="1.0" encoding="utf-8" ?>
1+
<?xml version="1.0" encoding="utf-8"?>
22
<configuration>
33
<startup>
4-
<supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.5" />
4+
<supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.6"/>
55
</startup>
6-
</configuration>
6+
</configuration>

SimpleDnsCrypt/Properties/Resources.Designer.cs

Lines changed: 12 additions & 12 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

SimpleDnsCrypt/Properties/Settings.Designer.cs

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

SimpleDnsCrypt/SimpleDnsCrypt.csproj

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,13 @@
99
<AppDesignerFolder>Properties</AppDesignerFolder>
1010
<RootNamespace>SimpleDnsCrypt</RootNamespace>
1111
<AssemblyName>SimpleDnsCrypt</AssemblyName>
12-
<TargetFrameworkVersion>v4.5</TargetFrameworkVersion>
12+
<TargetFrameworkVersion>v4.6</TargetFrameworkVersion>
1313
<FileAlignment>512</FileAlignment>
1414
<ProjectTypeGuids>{60dc8134-eba5-43b8-bcc9-bb4bc16c2548};{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}</ProjectTypeGuids>
1515
<WarningLevel>4</WarningLevel>
1616
<NuGetPackageImportStamp>
1717
</NuGetPackageImportStamp>
18+
<TargetFrameworkProfile />
1819
</PropertyGroup>
1920
<PropertyGroup Condition=" '$(Configuration)|$(Platform)' == 'Debug|AnyCPU' ">
2021
<PlatformTarget>AnyCPU</PlatformTarget>

SimpleDnsCrypt/ViewModels/BlockViewModel.cs

Lines changed: 25 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -616,22 +616,24 @@ public async void BuildDomainBlacklist()
616616
{
617617
try
618618
{
619-
//
620619
if (IsWorking) return;
621620
IsWorking = true;
622621
DomainBlacklistPathInfoString = LocalizationEx.GetUiString("blacklist_building_list",
623622
Thread.CurrentThread.CurrentCulture);
624623
var domainBlacklist = await GenerateDomainBlacklist(DomainBlacklist).ConfigureAwait(false);
625-
File.WriteAllLines(Path.Combine("data", Global.DomainBlacklistFile), domainBlacklist);
626-
DomainBlacklistPath = Path.Combine(Directory.GetCurrentDirectory(), "data", Global.DomainBlacklistFile);
627-
UpdateDomainBlacklistPathInfoString();
628-
if (DomainBlacklistPlugin)
624+
if (domainBlacklist != null)
629625
{
630-
SavePlugin();
626+
File.WriteAllLines(Path.Combine("data", Global.DomainBlacklistFile), domainBlacklist);
627+
DomainBlacklistPath = Path.Combine(Directory.GetCurrentDirectory(), "data", Global.DomainBlacklistFile);
628+
UpdateDomainBlacklistPathInfoString();
629+
if (DomainBlacklistPlugin)
630+
{
631+
SavePlugin();
632+
}
631633
}
632634
IsWorking = false;
633635
}
634-
catch (Exception)
636+
catch (Exception exception)
635637
{
636638
IsWorking = false;
637639
}
@@ -663,11 +665,19 @@ public void BuildAddressBlacklist()
663665

664666
private static async Task<string> FetchRemoteList(string requestUri)
665667
{
666-
using (var client = new HttpClient())
668+
try
669+
{
670+
using (var client = new HttpClient())
671+
{
672+
var getDataTask = client.GetStringAsync(requestUri);
673+
return await getDataTask.ConfigureAwait(false);
674+
}
675+
}
676+
catch (Exception)
667677
{
668-
var getDataTask = client.GetStringAsync(requestUri);
669-
return await getDataTask.ConfigureAwait(false);
678+
670679
}
680+
return null;
671681
}
672682

673683

@@ -703,8 +713,11 @@ public static async Task<List<string>> GenerateDomainBlacklist(DomainBlacklist d
703713
foreach (var remoteRule in domainBlacklist.RemoteRules)
704714
{
705715
var r = await FetchRemoteList(remoteRule.Rule).ConfigureAwait(false);
706-
var remote = ParseBlacklist(r);
707-
allNames.AddUnique(remote);
716+
if (r != null)
717+
{
718+
var remote = ParseBlacklist(r);
719+
allNames.AddUnique(remote);
720+
}
708721
}
709722

710723
return allNames;

0 commit comments

Comments
 (0)