oVirt / go-ovirt-client

An easy-to-use overlay for the Go SDK.
https://pkg.go.dev/github.com/ovirt/go-ovirt-client
BSD 3-Clause "New" or "Revised" License
11 stars 20 forks source link

TestBadURL bug: connection reset by peer #218

Open ghost opened 2 years ago

ghost commented 2 years ago

Describe the bug

TestBadURL fails with the following error:

new_test.go:97: the returned error was not an ENotAnOVirtEngine (generic_error: non-retryable error encountered while testing oVirt engine connection, giving up (failed to validate the connection (Post "https://example.com/ovirt-engine/sso/oauth/token": read tcp 10.1.0.122:60858->93.184.216.34:443: read: connection reset by peer)))
ghost commented 2 years ago

This is most likely due to using https://example.com as a test URL. This should be changed to use an embedded webserver.

arso commented 2 years ago

'example.com' can temporarily ban you if you use it too much ie. for automated integration tests. It happened to one of my old projects. Artur

On Mon, Jun 13, 2022 at 1:05 PM Janos Bonic @.***> wrote:

This is most likely due to using https://example.com as a test URL. This should be changed to use an embedded webserver.

— Reply to this email directly, view it on GitHub https://github.com/oVirt/go-ovirt-client/issues/218#issuecomment-1153778867, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACMF2BBKVEG3TA6CCGZJNLVO4IV7ANCNFSM5YT2ENAQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Artur Socha Senior Software Engineer, RHV Red Hat

ghost commented 2 years ago

Thanks, @arso I supposed that happened to us. It's not a difficult fix, we just have to do it.