Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

audio streaming header に対応する #188

Merged
merged 7 commits into from
Nov 7, 2024

Conversation

Hexa
Copy link
Contributor

@Hexa Hexa commented Nov 7, 2024

This pull request introduces support for handling audio streaming headers in the project. The main changes include adding a new configuration option, updating relevant configuration files, and implementing logic to process audio data with headers.

Key changes:

Configuration Updates:

  • Added audio_streaming_header setting to the Config struct in config.go to handle audio streaming headers.
  • Updated config_example.ini to include the new audio_streaming_header setting with a default value of false.

Documentation:

  • Updated CHANGES.md to document the addition of the audio_streaming_header setting.

Code Implementation:

  • Added readPacketWithHeader function in handler.go to process audio data with headers.
  • Modified the opus2ogg function in handler.go to use readPacketWithHeader when audio_streaming_header is enabled.

Imports:

  • Added the encoding/binary package import in handler.go to support the new header processing logic.
    This pull request introduces a new feature to handle audio streaming headers in the application. The key changes include updating the configuration to support this feature, modifying the handler to process audio data with headers, and adding a new function to read packets with headers.

Configuration Updates:

  • config.go: Added a new configuration option AudioStreamingHeader to handle audio streaming headers.
  • config_example.ini: Updated the example configuration file to include the new audio_streaming_header option.

Handler Modifications:

  • handler.go: Added a new function readPacketWithHeader to process audio data with headers.
  • handler.go: Updated the opus2ogg function to use the new readPacketWithHeader function if the AudioStreamingHeader configuration is enabled.

Documentation:

  • CHANGES.md: Documented the addition of the audio streaming header feature and its configuration.

@Hexa Hexa merged commit 3302a08 into develop Nov 7, 2024
1 check passed
@Hexa Hexa deleted the feature/support-streaming-header branch November 7, 2024 07:06
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