Skip to content

feat: per-peer MTU#2676

Merged
ShadauxCat merged 7 commits intodevelopfrom
feat/per_client_MTU
Aug 30, 2023
Merged

feat: per-peer MTU#2676
ShadauxCat merged 7 commits intodevelopfrom
feat/per_client_MTU

Conversation

@ShadauxCat
Copy link
Copy Markdown
Collaborator

@ShadauxCat ShadauxCat commented Aug 24, 2023

MTT-7202

fix: #2671

Changelog

  • Added: Added methods NetworkManager.SetPeerMTU and NetworkManager.GetPeerMTU to be able to set MTU sizes per-peer

Testing and Documentation

  • Includes unit tests.
  • No documentation changes or additions were necessary.

@ShadauxCat ShadauxCat requested a review from a team as a code owner August 24, 2023 20:32
-apparently GetValueOrDefault doesn't exist in CI?
Copy link
Copy Markdown
Member

@NoelStephensUnity NoelStephensUnity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and clean Kitty!

@ShadauxCat ShadauxCat enabled auto-merge (squash) August 30, 2023 17:03
@ShadauxCat ShadauxCat merged commit a981703 into develop Aug 30, 2023
@ShadauxCat ShadauxCat deleted the feat/per_client_MTU branch August 30, 2023 19:10
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

Successfully merging this pull request may close these issues.

MTU Value is shared across all peers

3 participants