Skip to content

Commit

Permalink
Make metadata optional. Missed additional review changes (#370)
Browse files Browse the repository at this point in the history
* Make metadata optional. Not enabled by default

* Turn variable into const

* Add missed review changes

* Bump RC version

---------

Co-authored-by: Shivank Dubey <[email protected]>
  • Loading branch information
Santiago-Souto and dubeyShivank authored Jun 10, 2024
1 parent 34e405e commit f7c69c2
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 8 deletions.
4 changes: 2 additions & 2 deletions packages/millicast-sdk/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/millicast-sdk/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@millicast/sdk",
"version": "0.1.45",
"version": "0.1.46-RC1",
"description": "SDK for building a realtime broadcaster using the Millicast platform.",
"keywords": [
"sdk",
Expand Down
4 changes: 1 addition & 3 deletions packages/millicast-sdk/src/Publish.js
Original file line number Diff line number Diff line change
Expand Up @@ -201,9 +201,7 @@ export default class Publish extends BaseWebRTC {
publisherData = await this.tokenGenerator()
// Set the iceServers from the publish data into the peerConfig
this.options.peerConfig.iceServers = publisherData?.iceServers
if (this.options.metadata) {
this.options.peerConfig.encodedInsertableStreams = supportsInsertableStreams
}
this.options.peerConfig.encodedInsertableStreams = this.options.metadata
} catch (error) {
logger.error('Error generating token.')
if (error instanceof FetchError) {
Expand Down
4 changes: 2 additions & 2 deletions packages/millicast-sdk/src/types/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1072,7 +1072,7 @@ declare module '@millicast/sdk' {
*/
absCaptureTime?: boolean;
/**
* - Enable metadata extraction if stream is compatible.
* - Enable metadata extraction. This feature is only supported with the H.264 codec.
*/
metadata?: boolean;
/**
Expand Down Expand Up @@ -1161,7 +1161,7 @@ declare module '@millicast/sdk' {
*/
bandwidth?: number;
/**
* - Enable metadata insertion if stream is compatible.
* - Enable metadata insertion. This feature is only supported with the H.264 codec.
*/
metadata?: boolean;
/**
Expand Down

0 comments on commit f7c69c2

Please sign in to comment.