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

PCMA not supported by WebRTCEndpoints #96

Open
3 tasks done
Mortega5 opened this issue Jan 13, 2025 · 1 comment
Open
3 tasks done

PCMA not supported by WebRTCEndpoints #96

Mortega5 opened this issue Jan 13, 2025 · 1 comment

Comments

@Mortega5
Copy link

Prerequisites

These are MANDATORY, otherwise the issue will be automatically closed.

Issue description

PCMA codec is not available for WebRTCEndpoint connections. I have been reading the source code, and it seems that PCMA was not included in the list of of supported codecs.

I suppose this is a bug and was not intentionally disabled.

KurentoWebRtcEndpointImpl WebRtcEndpointImpl.cpp:115:remove_not_supported_codecs_from_array:<kmswebrtcendpoint0> Removing not supported codec 'PCMA/8000'

Context

I'm using PCMA for RtpEndpoint, and I want to avoid transcoding by using the same codec (I believe this will improve performance).

How to reproduce?

  1. Enable PCMA codec on SdpEndpoint.config
    {
      "audioCodecs": [
        {
          "name": "PCMA/8000"
        },
        {
          "name": "PCMU/8000"
        }
      ],
      "numAudioMedias": 1,
      "numVideoMedias": 1,
      "videoCodecs": [
        {
          "name": "VP8/90000"
        },
        {
          "name": "H264/90000"
        }
      ]
    }
  1. Create a pipeline
  2. Create a WebRtcEndpoint

Expected & current behavior

It should support PCMA

Info about your environment

WebRTC streaming is relayed using a TURN (coturn

About Kurento Media Server

About your Application Server

  • Programming Language: Java
  • Kurento Client version: 7.1.0

About end-user clients

  • Device(s): any
  • OS(es): any
  • Browser(s): any

Run these commands

cat /etc/lsb-release
kurento-media-server --version
dpkg -l | grep -Pi 'kurento|kms-|gst.*1.5|nice'
@slabajo
Copy link
Member

slabajo commented Jan 15, 2025

We have made a fix for this on PR #99 after testing it should go merged on next release

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

No branches or pull requests

2 participants