Exposing routerId and transportId properties

vpalmisano · August 26, 2020, 11:41am

It will be useful exposing the routerId and the transportId internal properties in Producer and Consumer classes. What do you think?

github.com

versatica/mediasoup/blob/v3/src/Producer.ts#L228


		this._payloadChannel = payloadChannel;
		this._appData = appData;
		this._paused = paused;

		this._handleWorkerNotifications();
	}

	/**
	 * Producer id.
	 */
	get id(): string
	{
		return this._internal.producerId;
	}

	/**
	 * Whether the Producer is closed.
	 */
	get closed(): boolean
	{
		return this._closed;

github.com

versatica/mediasoup/blob/v3/src/Consumer.ts#L256


		this._producerPaused = producerPaused;
		this._score = score;
		this._preferredLayers = preferredLayers;

		this._handleWorkerNotifications();
	}

	/**
	 * Consumer id.
	 */
	get id(): string
	{
		return this._internal.consumerId;
	}

	/**
	 * Associated Producer id.
	 */
	get producerId(): string
	{
		return this._internal.producerId;

ibc · August 26, 2020, 11:48am

I don’t like exposing properties artificially. If you create a Producer you know in which Transport and Router you have created it, no need to ask later to the Producer to tell you that.

vpalmisano · August 26, 2020, 11:51am

Yes, but I need to save the transportId into another variable (or in appData), but it is already stored into the Producer/Consumer class _internal property.

ibc · August 26, 2020, 11:57am

Yep, but that’s custom usage or custom needs. You can store it into appData or into another objects your app holds.

Topic		Replies	Views
Was producer map removed? Off Topic	19	317	November 20, 2021
Listing producers/consumers associated with a given transport mediasoup libraries	3	444	April 9, 2020
Pipe stream from one mediasoup to another on different hosts mediasoup libraries	2	411	July 22, 2021
Keep track of workers, routers, transports, etc mediasoup libraries	10	987	December 2, 2019
Piping producers to other router with the same producer ID mediasoup libraries	5	607	August 21, 2020

Exposing routerId and transportId properties

Related Topics