Skip to content

Bump the nuget group with 3 updates #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 10, 2025
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/CHttpExecutor/CHttpExecutor.csproj
Original file line number Diff line number Diff line change
@@ -26,7 +26,7 @@
</ItemGroup>

<ItemGroup>
<PackageReference Include="JsonPath.Net" Version="2.1.0" />
<PackageReference Include="JsonPath.Net" Version="2.1.1" />
</ItemGroup>

</Project>
4 changes: 2 additions & 2 deletions src/CHttpExtension/CHttpExtension.csproj
Original file line number Diff line number Diff line change
@@ -11,8 +11,8 @@
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Microsoft.JavaScript.NodeApi" Version="0.9.1" PrivateAssets="none" />
<PackageReference Include="Microsoft.JavaScript.NodeApi.Generator" Version="0.9.1" />
<PackageReference Include="Microsoft.JavaScript.NodeApi" Version="0.9.3" PrivateAssets="none" />
<PackageReference Include="Microsoft.JavaScript.NodeApi.Generator" Version="0.9.3" />
</ItemGroup>

<ItemGroup>

Unchanged files with check annotations Beta

public class Data
{
public string Message { get; set; }

Check warning on line 76 in tests/TestWebApplication/Program.cs

GitHub Actions / build-and-deploy

Non-nullable property 'Message' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
}
public partial class Program { }
private readonly SslStreamCertificateContext? _serverCertificateContext;
private readonly X509Certificate2? _serverCertificate;
private readonly Func<CHttpConnectionContext, string?, X509Certificate2?>? _serverCertificateSelector;

Check warning on line 29 in src/CHttpServer/CHttpServer/HttpsConnectionMiddleware.cs

GitHub Actions / build-and-deploy

Field 'HttpsConnectionMiddleware._serverCertificateSelector' is never assigned to, and will always have its default value null
public HttpsConnectionMiddleware(Func<CHttpConnectionContext,Task> next, HttpsConnectionAdapterOptions options)
{
}
var sslDuplexPipe = CreateSslDuplexPipe(
context.Transport,

Check warning on line 73 in src/CHttpServer/CHttpServer/HttpsConnectionMiddleware.cs

GitHub Actions / build-and-deploy

Possible null reference argument for parameter 'transport' in 'DuplexPipeStreamAdapter<SslStream> HttpsConnectionMiddleware.CreateSslDuplexPipe(Stream transport, MemoryPool<byte> memoryPool)'.
context.Features.Get<IMemoryPoolFeature>()?.MemoryPool ?? MemoryPool<byte>.Shared);
var sslStream = sslDuplexPipe.Stream;
await sslStream.DisposeAsync();
return;
}
catch (IOException e)

Check warning on line 96 in src/CHttpServer/CHttpServer/HttpsConnectionMiddleware.cs

GitHub Actions / build-and-deploy

The variable 'e' is declared but never used
{
await sslStream.DisposeAsync();
return;
private readonly CHttpConnectionContext _context;
private readonly Stream _inputStream;
private readonly int _streamIdIndex;

Check warning on line 28 in src/CHttpServer/CHttpServer/Http2Connection.cs

GitHub Actions / build-and-deploy

Field 'Http2Connection._streamIdIndex' is never assigned to, and will always have its default value 0
private readonly HPackDecoder _hpackDecoder;
private byte[] _buffer;
{
errorCode = Http2ErrorCode.STREAM_CLOSED;
}
catch (Exception e)

Check warning on line 93 in src/CHttpServer/CHttpServer/Http2Connection.cs

GitHub Actions / build-and-deploy

The variable 'e' is declared but never used
{
}
finally
internal class SampleRequest
{
public string A { get; set; }

Check warning on line 68 in src/CHttpServer/TestCHttpServerApplication/Program.cs

GitHub Actions / build-and-deploy

Non-nullable property 'A' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
}
timeProvider.Advance(TimeSpan.FromMilliseconds(49));
Assert.False(waiting.IsCompleted);
timeProvider.Advance(TimeSpan.FromMilliseconds(1));
await waiting.WaitAsync(TimeSpan.FromSeconds(1));

Check warning on line 17 in tests/CHttp.Tests/AwaiterTests.cs

GitHub Actions / build-and-deploy

Calls to methods which accept CancellationToken should use TestContext.Current.CancellationToken to allow test cancellation to be more responsive. (https://xunit.net/xunit.analyzers/rules/xUnit1051)
}
}
var awaiter = new SyncedAwaiter(1);
var sut = new UploadThrottledStringContent(input, 1, awaiter);
using var ms = new MemoryStream();
await sut.CopyToAsync(ms);

Check warning on line 15 in tests/CHttp.Tests/Http/UploadThrottledStringContentTests.cs

GitHub Actions / build-and-deploy

Calls to methods which accept CancellationToken should use TestContext.Current.CancellationToken to allow test cancellation to be more responsive. (https://xunit.net/xunit.analyzers/rules/xUnit1051)
ms.Seek(0, SeekOrigin.Begin);
Assert.Equal(Encoding.UTF8.GetBytes(input), ms.ToArray());
Assert.Equal(1, awaiter.RemainingCount);
var awaiter = new SyncedAwaiter(1);
var sut = new UploadThrottledStringContent(input, 1, awaiter);
using var ms = new MemoryStream();
await sut.CopyToAsync(ms);

Check warning on line 28 in tests/CHttp.Tests/Http/UploadThrottledStringContentTests.cs

GitHub Actions / build-and-deploy

Calls to methods which accept CancellationToken should use TestContext.Current.CancellationToken to allow test cancellation to be more responsive. (https://xunit.net/xunit.analyzers/rules/xUnit1051)
ms.Seek(0, SeekOrigin.Begin);
Assert.Equal(Encoding.UTF8.GetBytes(input), ms.ToArray());
Assert.Equal(0, awaiter.RemainingCount);