Skip to content

Commit

Permalink
Merge pull request #92 from Slach/switch_to_ping_handler
Browse files Browse the repository at this point in the history
we need change `GET /` to `GET /ping` for Ping() function
  • Loading branch information
DoubleDi committed Mar 7, 2020
2 parents 1f18709 + 13ea952 commit e0d2ce7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions conn_go18.go
Expand Up @@ -16,8 +16,8 @@ func (c *conn) Ping(ctx context.Context) error {
if c.transport == nil {
return ErrTransportNil
}
// make request with empty body, response must be "Ok"
u := &url.URL{Scheme: c.url.Scheme, User: c.url.User, Host: c.url.Host, Path: "/"}
// make request with empty body, response must be "Ok.\n"
u := &url.URL{Scheme: c.url.Scheme, User: c.url.User, Host: c.url.Host, Path: "/ping"}
req, err := http.NewRequest(http.MethodGet, u.String(), nil)
if err != nil {
return err
Expand Down

0 comments on commit e0d2ce7

Please sign in to comment.