Skip to content

Commit 5d8db8d

Browse files
author
William Denton
committed
Remove multi targeting of net461
Elasticsearch.Net were previously hedging their bets on using HttpClient in Fullframework. They were was using HttpWebRequest and httpclient in net standard, there is no longer this compile split, so we can remove it from here too.
1 parent 3e5ad1e commit 5d8db8d

File tree

7 files changed

+5
-91
lines changed

7 files changed

+5
-91
lines changed

src/Elasticsearch.Net.Aws/ElasticSearch.Net.Aws.IntegrationTests/ElasticSearch.Net.Aws.IntegrationTests.csproj

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

33
<PropertyGroup>
4-
<TargetFrameworks>netcoreapp2.0;net461</TargetFrameworks>
4+
<TargetFrameworks>netcoreapp2.0</TargetFrameworks>
55
<AssemblyName>ElasticSearch.Net.Aws.IntegrationTests</AssemblyName>
66
<PackageId>ElasticSearch.Net.Aws.IntegrationTests</PackageId>
77
<GenerateRuntimeConfigurationFiles>true</GenerateRuntimeConfigurationFiles>
@@ -22,11 +22,6 @@
2222
<PackageReference Include="NUnit3TestAdapter" Version="3.8.0" />
2323
</ItemGroup>
2424

25-
<ItemGroup Condition=" '$(TargetFramework)' == 'net461' ">
26-
<Reference Include="System" />
27-
<Reference Include="Microsoft.CSharp" />
28-
</ItemGroup>
29-
3025
<ItemGroup>
3126
<Service Include="{82a7f48d-3b50-4b1e-b82e-3ada8210c358}" />
3227
</ItemGroup>

src/Elasticsearch.Net.Aws/ElasticSearch.Net.Aws.Tests/ElasticSearch.Net.Aws.Tests.csproj

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<Project Sdk="Microsoft.NET.Sdk">
22

33
<PropertyGroup>
4-
<TargetFrameworks>netcoreapp2.0;net461</TargetFrameworks>
4+
<TargetFrameworks>netcoreapp2.0</TargetFrameworks>
55
<AssemblyName>ElasticSearch.Net.Aws.Tests</AssemblyName>
66
<PackageId>ElasticSearch.Net.Aws.Tests</PackageId>
77
<GenerateRuntimeConfigurationFiles>true</GenerateRuntimeConfigurationFiles>
@@ -20,18 +20,10 @@
2020
<PackageReference Include="NEST" Version="6.0.1" />
2121
<PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
2222
<PackageReference Include="NUnit3TestAdapter" Version="3.8.0" />
23-
</ItemGroup>
24-
25-
<ItemGroup Condition=" '$(TargetFramework)' == 'netcoreapp2.0' ">
2623
<PackageReference Include="Elasticsearch.Net" Version="6.0.1" />
2724
<PackageReference Include="System.Diagnostics.TraceSource" Version="4.3.0" />
2825
</ItemGroup>
2926

30-
<ItemGroup Condition=" '$(TargetFramework)' == 'net461' ">
31-
<Reference Include="System" />
32-
<Reference Include="Microsoft.CSharp" />
33-
</ItemGroup>
34-
3527
<ItemGroup>
3628
<Service Include="{82a7f48d-3b50-4b1e-b82e-3ada8210c358}" />
3729
</ItemGroup>

src/Elasticsearch.Net.Aws/ElasticSearch.Net.Aws.Tests/SignUtilTests.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,19 +18,11 @@ public void SetUp()
1818
{
1919
var encoding = new UTF8Encoding(false);
2020
_sampleBody = encoding.GetBytes("Action=ListUsers&Version=2010-05-08");
21-
#if NETCOREAPP
2221
var request = new System.Net.Http.HttpRequestMessage(System.Net.Http.HttpMethod.Post, "https://iam.amazonaws.com/");
2322
request.Content = new System.Net.Http.ByteArrayContent(_sampleBody);
2423
request.Content.Headers.TryAddWithoutValidation("Content-Type", "application/x-www-form-urlencoded; charset=utf-8");
2524
request.Headers.TryAddWithoutValidation("X-Amz-Date", "20110909T233600Z");
2625
_sampleRequest = new HttpRequestMessageAdapter(request);
27-
#elif NET461
28-
var request = System.Net.WebRequest.CreateHttp("https://iam.amazonaws.com/");
29-
request.Method = "POST";
30-
request.ContentType = "application/x-www-form-urlencoded; charset=utf-8";
31-
request.Headers["X-Amz-Date"] = "20110909T233600Z";
32-
_sampleRequest = new HttpWebRequestAdapter(request);
33-
#endif
3426
}
3527

3628
[Test]

src/Elasticsearch.Net.Aws/Elasticsearch.Net.Aws/AwsHttpConnection.cs

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -43,21 +43,12 @@ public AwsHttpConnection() : this(
4343
{
4444
}
4545

46-
#if !NETSTANDARD
47-
protected override System.Net.HttpWebRequest CreateHttpWebRequest(RequestData requestData)
48-
{
49-
var request = base.CreateHttpWebRequest(requestData);
50-
SignRequest(new HttpWebRequestAdapter(request), requestData);
51-
return request;
52-
}
53-
#else
5446
protected override HttpRequestMessage CreateHttpRequestMessage(RequestData requestData)
5547
{
5648
var request = base.CreateHttpRequestMessage(requestData);
5749
SignRequest(new HttpRequestMessageAdapter(request), requestData);
5850
return request;
5951
}
60-
#endif
6152

6253
private void SignRequest(IRequest request, RequestData requestData)
6354
{

src/Elasticsearch.Net.Aws/Elasticsearch.Net.Aws/Elasticsearch.Net.Aws.csproj

Lines changed: 1 addition & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
<PropertyGroup>
44
<VersionPrefix>6.1.0</VersionPrefix>
5-
<TargetFrameworks>net461;netstandard2.0</TargetFrameworks>
5+
<TargetFrameworks>netstandard2.0</TargetFrameworks>
66
<AssemblyName>Elasticsearch.Net.Aws</AssemblyName>
77
<PackageId>Elasticsearch.Net.Aws</PackageId>
88
<PackageVersion>6.1.0</PackageVersion>
@@ -29,17 +29,6 @@
2929
<PackageReference Include="AWSSDK.Core" Version="3.3.17.9" />
3030
<PackageReference Include="Elasticsearch.Net" Version="6.*" />
3131
<PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
32-
</ItemGroup>
33-
34-
<ItemGroup Condition=" '$(TargetFramework)' == 'net461' ">
35-
<Reference Include="System.Configuration" />
36-
<Reference Include="System.Net.Http" />
37-
<Reference Include="System.Web" />
38-
<Reference Include="System" />
39-
<Reference Include="Microsoft.CSharp" />
40-
</ItemGroup>
41-
42-
<ItemGroup Condition=" '$(TargetFramework)' == 'netstandard2.0' ">
4332
<PackageReference Include="Microsoft.AspNetCore.WebUtilities" Version="1.1.2" />
4433
</ItemGroup>
4534

src/Elasticsearch.Net.Aws/Elasticsearch.Net.Aws/HttpWebRequestAdapter.cs

Lines changed: 0 additions & 42 deletions
This file was deleted.

src/Elasticsearch.Net.Aws/Elasticsearch.Net.Aws/SignV4Util.cs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -157,18 +157,15 @@ private static void WriteSignedHeaders(Dictionary<string, string> canonicalHeade
157157
}
158158
}
159159

160-
#if !NETSTANDARD
161-
private static NameValueCollection ParseQueryString(string query) =>
162-
System.Web.HttpUtility.ParseQueryString(query);
163-
#else
160+
164161
private static NameValueCollection ParseQueryString(string query) =>
165162
Microsoft.AspNetCore.WebUtilities.QueryHelpers.ParseQuery(query)
166163
.Aggregate(new NameValueCollection(), (col, kv) =>
167164
{
168165
kv.Value.ToList().ForEach(v => col.Add(kv.Key, v));
169166
return col;
170167
});
171-
#endif
168+
172169
public static string GetCanonicalQueryString(this Uri uri)
173170
{
174171
if (string.IsNullOrWhiteSpace(uri.Query)) return string.Empty;

0 commit comments

Comments
 (0)