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

Неверное описание + переработка механизма выдачи слотов по IP адресам #1042

Closed
pavel-pimenov opened this issue Aug 24, 2015 · 3 comments

Comments

@pavel-pimenov
Copy link
Owner

From tret2...@gmail.com on April 24, 2013 11:43:30

http://flylinkdc.com/dokuwiki/doku.php?id=ru:slots В описании указано что это выдача слотов по IP адресам, но судя по коду и по названию меток настроек if (BOOLSETTING(EXTRA_SLOT_BY_IP)) это всё таки выдача дополнительных слотов.

Так же вопрос необходимости галки выдачи доп слота своей сети. Алгоритм её такой - берётся свой IP из настроек хаба, если его нет из настроек клиента, и сверяется входит ли IP запрашивающего подключение клиента с нашим в одну сеть по маске 24 (кому трудно с масками это проверка только первых трёх цифры IP). Думаю такую проверку не имеет смысла делать, пользователь сам вполне может сразу прописать все адреса (раз он туда начал записывать адреса, он знает что делает) и не не всегда с моей сети маска 24, есть и 19 и 20... Тем более что реализация такой проверки сделана не оптимально.

Original issue: http://code.google.com/p/flylinkdc/issues/detail?id=1005

@pavel-pimenov
Copy link
Owner Author

From tret2...@gmail.com on April 24, 2013 03:26:07

Owner: tret2...@gmail.com

@pavel-pimenov
Copy link
Owner Author

From tret2...@gmail.com on April 24, 2013 03:54:52

This issue was closed by revision r13734 .

Status: Fixed

@pavel-pimenov
Copy link
Owner Author

From tret2...@gmail.com on April 24, 2013 03:55:46

Summary: Неверное описание + переработка механизма выдачи слотов по IP адресам (was: Неверное описание + переработка механизма выдачи сотов по IP адресам)

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

No branches or pull requests

1 participant