-
Notifications
You must be signed in to change notification settings - Fork 7
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
Comments
Спасибо за предложение, но, честно говоря, вариант не очень. Только лишний шум. Если у меня настроен вариант "с заголовком", ни к чему мне знать, сколько будет без него, и наоборот. Дополнительно, с заголовками в icmp всё сложно. Их два - icmp-заголовок (8 байт) и ip-заголовок (от 20 байт и больше). А в разных реализациях размеры считают по-разному - где-то без заголовков, где-то с одним, где-то с обоими. И вот это вот предсказание только запутывать больше будет. К тому же прогнозировать размер IP-заголовка нет возможности из-за "и больше", чаще всего это 20 байт, но кое-где бывает больше, и приложение не может это контролировать. Да и не вижу я, чтобы у пользователей какие-то проблемы были с размерами пакетов. Разве что вопросы были после того, как вынужденно пришлось убрать попытку считать IP-заголовок, всё. PS: Кстати, шрифт такого размера нельзя делать, у нас всё-таки не кредитный договор =) Так что тут две строки как минимум. |
ну тогда в описание или еще куда, добавьте информацию как правильно считать размер ICMP пакета. |
Можно указать размер пакета ICMP в учетом заголовка и без заголовка в описани?
Так проще и понятнее настраивать будет. Вот пример:
The text was updated successfully, but these errors were encountered: