Skip to content

fix: fix sipflow rtp recording#120

Merged
yeoleobun merged 1 commit intorestsend:mainfrom
yeoleobun:main
Feb 4, 2026
Merged

fix: fix sipflow rtp recording#120
yeoleobun merged 1 commit intorestsend:mainfrom
yeoleobun:main

Conversation

@yeoleobun
Copy link
Collaborator

This pull request contains following fix:

  • Standard G722 does not support stereo, g722 recording does not work.
    • decode g722 to pcmu
  • Remove the raw_packet.clone(), get the packet before transcoding.
  • Since sipflow rtp decode need the header, use packet.mashal() instead of packet.payload, and useRtpPacket::parse instead of manually parse which assume fixed size header.
  • Using zstd magic number predicate instead of compare the size. (the compressed data may have same size with the original)

@yeoleobun yeoleobun merged commit 8ac027e into restsend:main Feb 4, 2026
1 check 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.

1 participant