search

tastytrade / tastytrade-api-js

Javascript sdk for the tastytrade api
MIT License
63 stars 34 forks source link

Catch any errors when instantiating https.Agent #24

Closed dmoss18 closed 4 months ago

dmoss18 commented 5 months ago

https://github.com/tastytrade/tastytrade-api-js/issues/18

Browsers should adhere to the 1.2 minimum TLS version, so we'll just catch any errors with this and move on.

dmoss18 commented 5 months ago

@osdevisnot I took it out completely. Let me know if you see any more warnings. If it looks good, I'll merge this and publish a new version.

osdevisnot commented 5 months ago

Amazing, this works better now. I would recommend we merge it, but wait to cut a new version after #25 is resolved.