Skip to content

Improve join_accept packet handling#345

Merged
GooberRF merged 5 commits intomasterfrom
newjoinaccept
Apr 7, 2026
Merged

Improve join_accept packet handling#345
GooberRF merged 5 commits intomasterfrom
newjoinaccept

Conversation

@GooberRF
Copy link
Copy Markdown
Owner

@GooberRF GooberRF commented Apr 7, 2026

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Alpine Faction’s join_accept packet extension to be more forward-compatible and safer to parse by introducing a footer-based tail format for newer clients while retaining legacy compatibility.

Changes:

  • Added patch/type version bytes and explicit packing/size assertion for AlpineFactionJoinAcceptPacketExt.
  • Send footer-based AF tail for AF 1.3+ clients; keep legacy raw-append for older clients.
  • Added a shared parser that can detect and parse either footer-based or legacy join_accept extensions; extended AlpineFactionServerInfo with patch/type fields.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
game_patch/multi/network.h Extends and hardens the on-wire join_accept AF extension struct (packing, version fields, defaults, size assert).
game_patch/multi/network.cpp Switches send format based on client version and adds dual-format parsing logic for received join_accept.
game_patch/multi/multi.h Stores additional AF server version info (patch/type) parsed from join_accept.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GooberRF GooberRF marked this pull request as ready for review April 7, 2026 04:29
@GooberRF GooberRF merged commit 4fab338 into master Apr 7, 2026
4 checks passed
@GooberRF GooberRF deleted the newjoinaccept branch April 7, 2026 04:29
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.

2 participants