search

odnoklassniki / ok-android-sdk

Android SDK & samples for native apps integrated with OK.RU
Apache License 2.0
20 stars 28 forks source link

OkTokenRequestListener #3

Closed hram closed 8 years ago

hram commented 8 years ago

В документации на SDK сказано "Для обработки авторизации необходимо, чтобы активити реализовало интерфейс OkTokenRequestListener (методы onSuccess, onCancel и onError)"

В SDK не нашел OkTokenRequestListener

lintax commented 8 years ago

Можно посмотреть пример, приложенный в https://github.com/odnoklassniki/ok-android-sdk/tree/master/odnoklassniki-android-sdk-example А документация на apiok - да, несколько устарела

thefilin commented 8 years ago

Действительно, в новой версии SDK нет onCancel. Есть предложение, как разделить неудачные кейсы OAuth между onError и onCancel ?

Например, сейчас, если отказаться выдавать пермиссии приложению, происходит редирект с параметром error, также, если есть проблема в настройках (запрашивается response_type=token, а у приложения не включен клиентский OAuth), то приходит редирект с тем же error, но другим текстом.

lintax commented 8 years ago

А чем должны различаться обработки ошибки и отмена? Суть с точки зрения API схожа - приложению отказано в доступе под текущим пользователем (причем та же отмена может быть на первом окне, а может при применении пермиссий) - или же спам система, например, отменила возможность заходить этим пользователем или в это приложение. Какой практический смысл различать эти состояния? Как мне кажется, потребность в такой настройке низка, и может быть достигнута обработкой определенной ошибки.

hram commented 8 years ago

@lintax как обработать ошибку чтобы узнать отмену?

lintax commented 8 years ago

Просьба предоставить конкретный естественный пример, когда типичной игре/приложению надо различать события - ошибка и отменил пользователь.

hram commented 8 years ago

Пример есть. Конкретный. Предоставить не могу. Требования заказчика. Отмена и ошибка это разные сценарии. С этим думаю сложно спорить. В документации написано что их можно различать. Заказчик захотел их различать.

lintax commented 8 years ago

Если сценария нет - есть обходное решение - посмотреть как называется ошибка, которая возвращается в случае отмены и проверять на то вернулась ли она или нет

hram commented 8 years ago

Пустой спор какой то. Это разные действия. Ошибка авторизации и отмена авторизации. Заказчик хочет их различать и это его право. А подвязываться на текст сообщения об ошибке это как? Действительно думаете что это норма? А если вы текст поменяете? А если текст ошибки на разных языках будет отличаться? Странно такое слышать от одноклассников.

hram commented 8 years ago

@thefilin прошу о помощи

lintax commented 8 years ago

Такой функциональности нет и не планируется. Если будет естественный сценарий необходимости то можно рассматривать расширения API в рамках развития oauth.

On Thu, Jun 2, 2016, 15:54 Evgeny Hramov notifications@github.com wrote:

@thefilin https://github.com/thefilin прошу о помощи

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/odnoklassniki/ok-android-sdk/issues/3#issuecomment-223282550, or mute the thread https://github.com/notifications/unsubscribe/ALUUB-f58jlnhUvyRb8I2zPBb9U2fzKrks5qHtKbgaJpZM4H_ExD .