Skip to content
This repository has been archived by the owner on Jan 13, 2021. It is now read-only.

Ping does not work correctly #412

Open
PrimozGodec opened this issue May 24, 2019 · 3 comments
Open

Ping does not work correctly #412

PrimozGodec opened this issue May 24, 2019 · 3 comments

Comments

@PrimozGodec
Copy link
Contributor

PrimozGodec commented May 24, 2019

Issue

Ping function from Hyper does not check the real state of the connection. In my case, I make a connection to the server and then I switch off the internet connection. When I ping after switching off the internet connection ping is still successful. I went through the Hyper implementation and found out that ping function does not check whether the response (pong) comes back from the server. It just sends a request. Am I correct?

An example:

from hyper import HTTP20Connection
conn = HTTP20Connection('api.garaza.io:443',force_proto='h2')
# here I swithc off internet connection
conn.ping(bytes(8))

Am I doing anything wrong? Should I check the response separately?

@PrimozGodec
Copy link
Contributor Author

@Lukasa can you give me just a hint on how to correct the issue or how to correctly use the ping function.

@PrimozGodec
Copy link
Contributor Author

@njsmith, @sethmlarson, and @tomchristie I would be extremely happy if you give me a hint on how to solve the issue. It is a functionality that needs to work but I do not know how to start solving the issue since I do not understand the library exactly. I would be also happy if someone confirms that it is an issue.

@tomchristie
Copy link

Sorry, can't help here - I've not dug into it myself. (We likely won't want httpx to send outgoing ping messages, or expose that level of functionality in a public interface.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants