search

tagomoris / presto-client-node

Distributed query engine Presto client library for node.js
MIT License
125 stars 57 forks source link

SSL redirect failing #33

Closed souryabharath closed 4 years ago

souryabharath commented 4 years ago

When I tried to connect to presto server using the presto-client library with SSL (cert and key info) and run execute query the REST call to /v1/statement works and when the client tried to hit the "nextUri" endpoint, the server returns 401 due to missing certificate information while making the call.

This PR fixes that issue. Please review and let me know.

https://github.com/tagomoris/presto-client-node/compare/master...souryabharath:patch-1

tagomoris commented 4 years ago

Just for note: #34

tagomoris commented 4 years ago

@souryabharath I've created a different change to fix this problem, as the branch apply-ssl-options-in-fetching-nexturi. Can you fetch that branch and check it if it actually fixes your problem?

souryabharath commented 4 years ago

@tagomoris The new changes fixed the redirect issue.

tagomoris commented 4 years ago

35 fixes this issue.