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

limitation period #164

Closed
vlab97 opened this issue Oct 19, 2022 · 8 comments
Closed

limitation period #164

vlab97 opened this issue Oct 19, 2022 · 8 comments
Labels
question Further information is requested

Comments

@vlab97
Copy link

vlab97 commented Oct 19, 2022

Problem details

It happens that the bot incorrectly sets other dates. in the config file, a period of 1 day is set, and it blocks for 1 year, forever, and so on. Is it possible to set an additional check for the limitation period?
6D746035-6BCD-4F2D-9089-26E75B160E22

Steps to reproduce

1 entering the chat of a new member
2 imposition by the bot of an incorrect period for blocking actions

Expected behavior

There should have been a 1 day limit.

Actual behavior

The screenshot has a 365 day limit.

Project version

1.7.0

Platform

Microsoft Windows

OS version

Windows 10

Python version

.Exe

Installation type

EXE installer

Logs

No response

@vlab97 vlab97 added the bug Something isn't working label Oct 19, 2022
@xvitaly
Copy link
Owner

xvitaly commented Oct 19, 2022

Did you used /restrict 1?

@vlab97
Copy link
Author

vlab97 commented Oct 19, 2022

Did you used /restrict 1?

No. Automatic restriction for new chat participants is used.

Can we switch to Russian? 😁

@vlab97
Copy link
Author

vlab97 commented Oct 19, 2022

ecasbot.json

"bantime": 86400,

@xvitaly
Copy link
Owner

xvitaly commented Oct 19, 2022

Can we switch to Russian?

OK.

"bantime": 86400,

Если никнейм пользователя набирает больше баллов, чем указано в nickgoal, делённое на 2, то его аккаунт ограничивается на 366 дней (максимум для Telegram Bot API).

Т.е. можно просто nickgoal изменить с 100 до 200 например, либо использовать любое другое желаемое значение.

@vlab97
Copy link
Author

vlab97 commented Oct 19, 2022

Can we switch to Russian?

OK.

"bantime": 86400,

Если никнейм пользователя набирает больше баллов, чем указано в nickgoal, делённое на 2, то его аккаунт ограничивается на 366 дней (максимум для Telegram Bot API).

Т.е. можно просто nickgoal изменить с 100 до 200 например, либо использовать любое другое желаемое значение.

В консоли ник набрал 50 баллов
INFO:ecasbot:Новый пользователь Игорь (-) вступил в чат - - (-). Баллы: 50.

nickgoal был выставлен на 100, по вашему совету увеличу значение в 2 раза. А дальше посмотрим. Спасибо

Просто странно, что даже по логу ник набрал 50/100 баллов и ушел в максимальную блокировку по сроку

@xvitaly
Copy link
Owner

xvitaly commented Oct 19, 2022

Просто странно, что даже по логу ник набрал 50/100 баллов и ушел в максимальную блокировку по сроку

Там алгоритм такой: если аккаунт набирает полный nickgoal (100 по умолчанию) или больше, то бот его сразу банит и выкидывает из чата, а если только половину (от 50 до 99 по умолчанию), то просто ограничивает на 366 дней, т.к. считает подозрительным.

@xvitaly xvitaly added question Further information is requested and removed bug Something isn't working labels Oct 19, 2022
@xvitaly xvitaly closed this as completed Oct 19, 2022
@vlab97
Copy link
Author

vlab97 commented Oct 19, 2022

Спасибо. Так же заметил, что бот не регистрирует отправленные команды от имени чата, как админа. То есть если админ анонимен, то команды /mute и прочее не работают для него.

@xvitaly
Copy link
Owner

xvitaly commented Oct 19, 2022

То есть если админ анонимен, то команды /mute и прочее не работают для него.

Логично ибо через Telegram Bot API нельзя определить администратора, который инициировал данное действие. Это можно достаточно легко исправить если в функцию проверки действий добавить и собственный ID чата, но нужно проверить для начала не будет ли это являться угрозой безопасности. Создал новый тикет #165 по данной проблеме.

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

No branches or pull requests

2 participants