Нужно ли делать $USER->Logout() для API скриптов? Нашёл в одном

Question

Нужно ли делать $USER->Logout() для API скриптов? Нашёл в одном

из них запуск действий от имени admin и в конце это разлогинивание. При этом в логах PHP это фиксируется ошибкой:

"PHP message: PHP Fatal error: Uncaught RuntimeException: Could not start session because headers have already been sent."

Похоже что пытается вывести HTML с формой логина ))

Мне кажется, что окончание PHP скрипта лучше любого разлогинивания, но раз мои предшественники это сделали, то может в этом был какой-то сакральный смысл?

#1c #bitrix #cms #programming #russian #software

0

30.06.2021

13 ответов

16 просмотров

Константин Салахутдинов Автор вопроса

Alexander Ostrovsky
если у тебя это в консольной сессии делается (или ...

в консольной сессии же нет браузера )

0

30.06.2021

Alexander Ostrovsky

Константин Салахутдинов
в консольной сессии же нет браузера )

в консольной нет, но я написал, что если можно это из браузера запустить, например прямым вызовом скрипта. да, там не в консольной, про пользовательскую сессию речь.

0

30.06.2021

Andrei Nikolaev

Нет, не нужно logout'ить, ровно как и авторизовывать тоже НЕ НУЖНО.

0

30.06.2021

Константин Салахутдинов Автор вопроса

Alexander Ostrovsky
в консольной нет, но я написал, что если можно это...

Понял. Но вроде там всё cli-шное. Проблема была в ошибке в логах PHP — в момент исполнения $USER->Logout() происходит ошибка: PHP message: PHP Fatal error: Uncaught RuntimeException: Could not start session because headers have already been sent.

0

30.06.2021

Andrei Nikolaev

Константин Салахутдинов
Понял. Но вроде там всё cli-шное. Проблема была в ...

Есть, в бетах :) Не production-ready.

0

30.06.2021

Константин Салахутдинов Автор вопроса

Andrei Nikolaev
Нет, не нужно logout'ить, ровно как и авторизовыва...

А зачем это могли делать? Чтобы при создании новых сущностей (сделки, счёта и тд) подставлялся верный автор?

0

30.06.2021

Alexander Ostrovsky

Константин Салахутдинов
А зачем это могли делать? Чтобы при создании новых...

да, возможно не знали как работать с сущностями. как умели - так и сделали. может с правами какой косяк.

0

30.06.2021

Andrei Nikolaev

Константин Салахутдинов
А зачем это могли делать? Чтобы при создании новых...

Нет, потому что не знали как можно отключить контроль прав.

0

30.06.2021

Константин Салахутдинов Автор вопроса

Andrei Nikolaev
Нет, потому что не знали как можно отключить контр...

Это который — 'CHECK_PERMISSIONS' => 'N' ?

0

30.06.2021

Andrei Nikolaev

Константин Салахутдинов
Это который — 'CHECK_PERMISSIONS' => 'N' ?

Это только на getList'ы, для создания нужно в конструктор CCrmDeal false передавать или в options пользователя от которого выполняется действие

0

30.06.2021

Константин Салахутдинов Автор вопроса

Andrei Nikolaev
Это только на getList'ы, для создания нужно в конс...

А можно пример или где почитать про это на будущее

0

30.06.2021

rix

Константин Салахутдинов
А можно пример или где почитать про это на будущее

https://bxapi.ru/src/?module_id=crm&name=CCrmDeal::__construct

0

30.06.2021

Alexander Ostrovsky · Accepted Answer

если у тебя это в консольной сессии делается (или может делаться), то в результате (без разлогина) в браузере останется админская сессия с полным доступом. ну и тут надо логику смотреть, очень мало вариантов когда надо в скрипте логиниться.

208 похожих чатов

Нужно ли делать $USER->Logout() для API скриптов? Нашёл в одном

13 ответов

Похожие вопросы