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

Добавить размер ICMP пакета в описание #19

Open
OmlineEditor opened this issue Aug 2, 2021 · 2 comments
Open

Comments

@OmlineEditor
Copy link

Можно указать размер пакета ICMP в учетом заголовка и без заголовка в описани?
Так проще и понятнее настраивать будет. Вот пример:

size

@impalex
Copy link
Owner

impalex commented Aug 2, 2021

Спасибо за предложение, но, честно говоря, вариант не очень.

Только лишний шум. Если у меня настроен вариант "с заголовком", ни к чему мне знать, сколько будет без него, и наоборот.

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

Да и не вижу я, чтобы у пользователей какие-то проблемы были с размерами пакетов. Разве что вопросы были после того, как вынужденно пришлось убрать попытку считать IP-заголовок, всё.

PS: Кстати, шрифт такого размера нельзя делать, у нас всё-таки не кредитный договор =) Так что тут две строки как минимум.

@OmlineEditor
Copy link
Author

ну тогда в описание или еще куда, добавьте информацию как правильно считать размер ICMP пакета.
Я немного мучалась пока настроила, было не понятно что за размер и как его считает ваша программа и iptables на VPS.

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

No branches or pull requests

2 participants