Feat: Add repeater hop limit specifically for advertisement packets#2408
Open
Steve-7acc wants to merge 4 commits intomeshcore-dev:mainfrom
Open
Feat: Add repeater hop limit specifically for advertisement packets#2408Steve-7acc wants to merge 4 commits intomeshcore-dev:mainfrom
Steve-7acc wants to merge 4 commits intomeshcore-dev:mainfrom
Conversation
Add advert_max_hops
Added advert.max.hops
Added condition for advert.max.hops
Fixed typo
Steve-7acc
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR introduces a user-defined hop limit specifically for advertisement packets, similar to the existing flood.max implementation, using a new command set/get advert.max.hops.
Currently, our network is seeing a high volume of advertisement traffic travelling excessive distances (40+ hops), which consumes significant airtime without providing a realistic contact or path due to increased probability of collision.
As shown in the attached messaging stats (credit: Mopatops), advertisement packets are accounting for 55% of total Air Time. Many of these packets originate from nodes far beyond a useful range. This creates additional traffic on the mesh and prevents messages from getting through.
Instead of lowering the global flood.max (which would impact all traffic), this change targets only advertisement packets, allows users to set a specific threshold for these packets and drops adverts that exceed this hop count while allowing normal messages and DMs to propagate normally.