search

messagebird / messagebird-nodejs

The open source Node.js client for MessageBird's REST API
https://www.messagebird.com/developers
105 stars 44 forks source link

Verify token method #2

Closed fvdm closed 9 years ago

fvdm commented 9 years ago

What is the preferred method name for token verification?

Verify now has read, create, delete and verify, but messagebird.verify.verify is not pretty.

samwierema commented 9 years ago

Yes, not very pretty. The alternative would be to leave out verify, and put it under read with an optional token (which means it will verify).

For now, let's go with the ugly messagebird.verify.verify option.

fvdm commented 9 years ago

How about messagebird.verify.token?

samwierema commented 9 years ago

I would leave it as messagebird.verify.verify. Even though it's a bit dubious, it does indicate what it does: verify. If you make it token, then that would be confusing as it could also (possibly) mean the creation of a token.

fvdm commented 9 years ago

OK I'll leave it like this.