search

prawn-cake / vk-requests

vk.com requests for humans. API library for vk.com
MIT License
158 stars 22 forks source link

Add Direct Authorization method #38

Closed proninyaroslav closed 5 years ago

proninyaroslav commented 5 years ago

https://github.com/prawn-cake/vk-requests/issues/37

proninyaroslav commented 5 years ago

Now VKSession gets bigger, I was thinking of some fresh ideas to split its' logic into more composable chunks, for instance use auth methods composition, instead of adding it to the same class.

Yes, the principles of authorization are similar and there is a feeling of code duplication. You can divide it into 3 separate authorizations, but Service key looks too small for that.

prawn-cake commented 5 years ago

I'll merge it and will test locally. Good job!