Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Авторизация по OAuth #40

Closed
leshchenko1979 opened this issue Aug 10, 2020 · 10 comments
Closed

Авторизация по OAuth #40

leshchenko1979 opened this issue Aug 10, 2020 · 10 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@leshchenko1979
Copy link
Owner

No description provided.

@leshchenko1979 leshchenko1979 added the enhancement New feature or request label Aug 10, 2020
@leshchenko1979
Copy link
Owner Author

  • Найти раздел доки про авторизацию по OAuth
  • Сделать тестовый аккаунт, где будет требоваться авторизация по OAuth

@leshchenko1979 leshchenko1979 added the help wanted Extra attention is needed label Oct 8, 2020
@leshchenko1979
Copy link
Owner Author

Можно взять за основу то, как сделано в https://github.com/yarbshk/pybitrix24

@leshchenko1979
Copy link
Owner Author

leshchenko1979 commented Jan 21, 2021

Можно сделать так:

В конструктор Bitrix() можно будет передать функцию-генератор, которая будет yield-ить токены.

Обращение к этой функции будет происходить каждый раз, когда от сервера будет приходить ошибка 403.

@leshchenko1979 leshchenko1979 self-assigned this Jan 21, 2021
@leshchenko1979
Copy link
Owner Author

Можно сделать так:

В конструктор Bitrix() можно будет передать функцию-генератор, которая будет yield-ить токены.

Обращение к этой функции будет происходить каждый раз, когда от сервера будет приходить ошибка 403.

Нет, слишком сложно. Нужен не генератор, а обычная функция. (Генератор нужен, когда внутри функции нужно сохранять состояние.)

leshchenko1979 added a commit that referenced this issue Jan 22, 2021
@leshchenko1979
Copy link
Owner Author

Возможно, имеет смысл в конструктор Bitrix() добавить параметр auth_func.

Эта функция, если она указана, будет вызываться перед первым запросом или когда сервер вернул ошибку 401. Если функция вернёт строку, то добавляем её как параметр auth в URL запроса. Если словарь, то считаем, что в словаре заголовки, которые нужно добавить к запросу.

Если вместо функции указана константа, то используем константу, как описано выше.

Заодно решит #140.

@leshchenko1979
Copy link
Owner Author

Тут подсказывают, что нужно взять реализацию отсюда: https://github.com/momoru-kun/PyBitrix/blob/main/pybitrix/pybitrix.py

@notrurs
Copy link

notrurs commented Jul 31, 2023

@leshchenko1979 Добрый день! По итогу возможность авторизации по OAuth ещё не добавили? Есть ли планы по этой фиче?

@leshchenko1979
Copy link
Owner Author

@leshchenko1979 Добрый день! По итогу возможность авторизации по OAuth ещё не добавили? Есть ли планы по этой фиче?

В #149 лежит веточка, которая ждёт тестирования. Если там все работает ОК, то можно выкатывать в мейн.

Не желаете ли протестировать?

@notrurs
Copy link

notrurs commented Aug 6, 2023

@leshchenko1979 на днях тестану, отпишусь)

@leshchenko1979
Copy link
Owner Author

Реализовано в #243

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants