Skip to content

Fix/additional tnl aper tag miss#27

Merged
ianchen0119 merged 3 commits into
free5gc:mainfrom
Alonza0314:fix/additional-tnl-aper-tag-miss
Oct 21, 2025
Merged

Fix/additional tnl aper tag miss#27
ianchen0119 merged 3 commits into
free5gc:mainfrom
Alonza0314:fix/additional-tnl-aper-tag-miss

Conversation

@Alonza0314

Copy link
Copy Markdown
Member
image

The missed aper tags is according to the image above....

@ss920386

Copy link
Copy Markdown

Hi @Alonza0314, the fix looks good to me.
Could you please add a unit test, or verify the aper tag fix with Wireshark?

@Alonza0314

Copy link
Copy Markdown
Member Author

Hi @ss920386 ,

I've added the unit test on checking the marshal and the unmarshal with these new aper reference value field is correct.

Also, here is the screenshot of both PathSwitchRequestTransfer and PathSwitchRequestAcknowledgeTransfer with wireshark:
request

requestAck

Thanks for your review!

@ianchen0119 ianchen0119 merged commit 333b38e into free5gc:main Oct 21, 2025
2 checks passed
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.

3 participants