Skip to content
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

IntegrationTests failed because of System.IO.IOException: The handshake failed due to an unexpected packet format. #184

Open
parhamsaremi opened this issue Nov 9, 2022 · 4 comments

Comments

@parhamsaremi
Copy link
Contributor

parhamsaremi commented Nov 9, 2022

link to the CI: https://github.com/parhamsaremi/geewallet/actions/runs/3427079272/jobs/5709641293

Unhandled Exception: System.AggregateException: One or more errors occurred. ---> System.IO.IOException: The handshake failed due to an unexpected packet format.
   at System.Net.Security.SslState.InternalEndProcessAuthentication(LazyAsyncResult lazyResult)
   at System.Net.Security.SslState.EndProcessAuthentication(IAsyncResult result)
   at System.Net.Security.SslStream.EndAuthenticateAsClient(IAsyncResult asyncResult)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncCoreLogic(IAsyncResult iar, Func`2 endFunction, Action`1 endAction, Task`1 promise, Boolean requiresSynchronization)
   --- End of inner exception stack trace ---
   at Microsoft.FSharp.Control.AsyncResult`1.Commit() in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\async.fs:line 349
   at Microsoft.FSharp.Control.AsyncPrimitives.RunSynchronouslyInCurrentThread[a](CancellationToken cancellationToken, FSharpAsync`1 computation) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\async.fs:line 870
   at Microsoft.FSharp.Control.AsyncPrimitives.RunSynchronously[T](CancellationToken cancellationToken, FSharpAsync`1 computation, FSharpOption`1 timeout) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\async.fs:line 890
   at Microsoft.FSharp.Control.FSharpAsync.RunSynchronously[T](FSharpAsync`1 computation, FSharpOption`1 timeout, FSharpOption`1 cancellationToken) in F:\workspace\_work\1\s\src\fsharp\FSharp.Core\async.fs:line 1153
   at GWallet.Backend.ServerManager.UpdateServersStats() in D:\a\geewallet\geewallet\src\GWallet.Backend\ServerManager.fs:line 213
   at Program.UpdateServersStats() in D:\a\geewallet\geewallet\src\GWallet.Frontend.Console\Program.fs:line 509
   at Program.main(String[] argv) in D:\a\geewallet\geewallet\src\GWallet.Frontend.Console\Program.fs:line 530
@knocte
Copy link
Member

knocte commented Nov 9, 2022

You got this in CI, paste the URL of that CI run.

@parhamsaremi
Copy link
Contributor Author

You got this in CI, paste the URL of that CI run.

sure but won't CI runs expire after some time? that's why I didn't paste it.

@knocte
Copy link
Member

knocte commented Nov 9, 2022

sure but won't CI runs expire after some time? that's why I didn't paste it.

But it's still valuable information until it expires. Also, pasting the CI link we know the crash happened in CI (as opposed to locally in your computer).

@parhamsaremi
Copy link
Contributor Author

sure but won't CI runs expire after some time? that's why I didn't paste it.

But it's still valuable information until it expires. Also, pasting the CI link we know the crash happened in CI (as opposed to locally in your computer).

right 👍

parhamsaremi added a commit to parhamsaremi/NOnion that referenced this issue Nov 9, 2022
Move TorGuard exception handling to a new module so that
every time we want to handle a new type of exception, we
won't be bothered by adding a lot of try/except expressions.

Fixes nblockchain#45
Fixes nblockchain/geewallet#184
parhamsaremi added a commit to parhamsaremi/NOnion that referenced this issue Nov 9, 2022
Move TorGuard exception handling to a new module so that
every time we want to handle a new type of exception, we
won't be bothered by adding a lot of try/except expressions.

Fixes nblockchain#45
Fixes nblockchain/geewallet#184
parhamsaremi added a commit to parhamsaremi/NOnion that referenced this issue Nov 11, 2022
Move TorGuard exception handling to a new module so that
every time we want to handle a new type of exception, we
won't be bothered by adding a lot of try/except expressions.
This code is necessary for fixing [1].

Fixes nblockchain#45
[1] nblockchain/geewallet#184
parhamsaremi added a commit to parhamsaremi/NOnion that referenced this issue Nov 14, 2022
Move TorGuard exception handling to a new module so that
every time we want to handle a new type of exception, we
won't be bothered by adding a lot of try/except expressions.
This code is necessary for fixing [1].

Fixes nblockchain#45
[1] nblockchain/geewallet#184
knocte pushed a commit to nblockchain/NOnion that referenced this issue Nov 14, 2022
Move TorGuard exception handling to a new module so that
every time we want to handle a new type of exception, we
won't be bothered by adding a lot of try/except expressions.

While we're at it, we wrap IOException (necessary for
fixing [1]).

Fixes #45
[1] nblockchain/geewallet#184
parhamsaremi added a commit to parhamsaremi/geewallet that referenced this issue Nov 14, 2022
parhamsaremi added a commit to parhamsaremi/geewallet that referenced this issue Nov 15, 2022
parhamsaremi added a commit to parhamsaremi/geewallet that referenced this issue Nov 15, 2022
parhamsaremi added a commit to parhamsaremi/geewallet that referenced this issue Nov 15, 2022
We have to use new for instanciating TorServiceHost since in the new
version of NOnion this type is IDisposable.

Fixes nblockchain#181
Fixes nblockchain#184
Fixes nblockchain#186
parhamsaremi added a commit to parhamsaremi/geewallet that referenced this issue Nov 15, 2022
We have to use new for instanciating TorServiceHost
since in the new version of NOnion this type is
IDisposable.

Fixes nblockchain#181
Fixes nblockchain#184
Fixes nblockchain#186
parhamsaremi added a commit to parhamsaremi/geewallet that referenced this issue Nov 15, 2022
We have to use new for instanciating TorServiceHost since in
the new version of NOnion this type is IDisposable.

Fixes nblockchain#181
Fixes nblockchain#184
Fixes nblockchain#186
knocte pushed a commit that referenced this issue Nov 16, 2022
We have to use new for instanciating TorServiceHost since in
the new version of NOnion this type is IDisposable.

Fixes #181
Fixes #184
Fixes #186
knocte pushed a commit that referenced this issue Nov 22, 2022
We have to use new for instanciating TorServiceHost since in
the new version of NOnion this type is IDisposable.

Fixes #181
Fixes #184
Fixes #186
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants