Return false in isAuthenticated instead of throwing

alpacahq / alpaca-ts

A TypeScript Node.js library for the https://alpaca.markets REST API and WebSocket streams.

ISC License

154 stars 42 forks source link

Return false in isAuthenticated instead of throwing #14

Closed lbstr closed 3 years ago

lbstr commented 3 years ago

In my opinion, isAuthenticated should return false instead of throwing. The user shouldn't have to try/catch around this. They should just call the method and get a boolean. Let me know if I'm missing something here.

117 commented 3 years ago

Agreed, this makes more sense.