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

Фикс обновления правил fw #25

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Elnadrion
Copy link

Здравствуйте,

Можете подтвердить, что $_SERVER['SCRIPT_URL'] тут не нужен?
Когда я сохранял настройки из админки - этот код выдает урл: https://mysite.ru/bitrix/admin/settings.php?spbc_remote_call_action=sfw_update&plugin_name=apbct&spbc_remote_call_token=7fdd0f46&firewall_updating_id=0&test=test и последующий HTTP запрос на этот урл выдает жопу: https://skr.sh/sRCLgTKXahA.jpg

С исправлением я получаю урл https://mysite.ru/?spbc_remote_call_action=sfw_update&plugin_name=apbct&spbc_remote_call_token=7fdd0f46&firewall_updating_id=0&test=test и "OK" в HTTP запросе

Без этого обновления правил фаервола через сохранение настроек модуля - не работает

@svfcode
Copy link
Member

svfcode commented Aug 14, 2024

@Elnadrion не удается воспроизвести проблему.
Код в который Вы предлагаете правки был написан более 3 лет назад и проблем не возникало.
По всей видимости проблема связана с локальными настройками.
Для дальнейших действий, пожалуйста, создайте обращение в нашей системе тикетов: https://cleantalk.org/my/support/open

@Elnadrion
Copy link
Author

Код в который Вы предлагаете правки был написан более 3 лет назад и проблем не возникало.

Я бы на вашем месте не стал бы списывать со счетов тот факт, что обновления для Битрикса выходят почти каждую неделю

Скорее всего на вашей установке отсутствует $_SERVER['SCRIPT_URL'], в таком случае он будет работать правильно. Подскажите, вы используете BitrixVM на вашей становке Битрикса и Apache в качестве веб сервера для бэка?

@svfcode
Copy link
Member

svfcode commented Aug 15, 2024

@Elnadrion, для тестов использовался LAMP стек.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants