search

Azure / azure-sdk-for-js

This repository is for active development of the Azure SDK for JavaScript (NodeJS & Browser). For consumers of the SDK we recommend visiting our public developer docs at https://docs.microsoft.com/javascript/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-js.
MIT License
2.07k stars 1.19k forks source link

[Communication calling] - selectMicrophone causes Uncaught (in promise) TypeError: this._telemetryLogManager is undefined #31461

Open rsanteri opened 1 day ago

rsanteri commented 1 day ago

Describe the bug After starting or joining room call and calling .selectMicrophone error triggers and microphone does not get changed.

Uncaught (in promise) TypeError: this._telemetryLogManager is undefined
    audioSourceChanged sdk.bundle.js:96
    emit sdk.bundle.js:95
    emit sdk.bundle.js:95
    _audioSourceChangedSub sdk.bundle.js:95
    raiseChanged sdk.bundle.js:72
    raiseEvents sdk.bundle.js:72
    raiseEvents sdk.bundle.js:72
    raiseChanged sdk.bundle.js:72
sdk.bundle.js:96:236229
    audioSourceChanged webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:96
    apply self-hosted:2292
    emit webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:95
    emit webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:95
    _audioSourceChangedSub webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:95
    raiseChanged webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    raiseEvents webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    forEach self-hosted:160
    raiseEvents webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    raiseChanged webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    notifyStreamChange webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    notifyChange webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    forEach self-hosted:1880
    notifyChange webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    onSelectedDevicesChanged webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    deviceManagerSubscriptions webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    _raiseEventImpl webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    raiseEvents webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    forEach self-hosted:160
    raiseEvents webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    _raiseEventImpl webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    raise webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    selectDevices webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    selectDevices webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    selectDevices webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:72
    selectMicrophone webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:96
    value webpack-internal:///./node_modules/@azure/communication-calling/dist/sdk.bundle.js:81
    get webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-stateful-client/src/DeviceManagerDeclarative.js:194
    withAsyncErrorTeedToState webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-stateful-client/src/CallContext.js:926
    InterpretGeneratorResume self-hosted:1420
    next self-hosted:1341
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-stateful-client/src/CallContext.js:25
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-stateful-client/src/CallContext.js:21
    withAsyncErrorTeedToState webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-stateful-client/src/CallContext.js:924
    onSelectMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-component-bindings/src/handlers/createCommonHandlers.js:129
    InterpretGeneratorResume self-hosted:1420
    next self-hosted:1341
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-component-bindings/src/handlers/createCommonHandlers.js:19
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-component-bindings/src/handlers/createCommonHandlers.js:15
    onSelectMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/calling-component-bindings/src/handlers/createCommonHandlers.js:125
    setMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:583
    InterpretGeneratorResume self-hosted:1420
    next self-hosted:1341
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:38
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:34
    setMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:582
    asyncTeeErrorToEventEmitter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:1084
    InterpretGeneratorResume self-hosted:1420
    next self-hosted:1341
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:38
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:34
    asyncTeeErrorToEventEmitter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:1082
    setMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:582
    InterpretGeneratorResume self-hosted:1420
    next self-hosted:1341
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:38
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:34
    setMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/adapter/AzureCommunicationCallAdapter.js:581
    onSelectMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/hooks/useHandlers.js:89
    InterpretGeneratorResume self-hosted:1420
    next self-hosted:1341
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/hooks/useHandlers.js:19
    __awaiter webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/hooks/useHandlers.js:15
    onSelectMicrophone webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-composites/src/composites/CallComposite/hooks/useHandlers.js:88
    onClick webpack-internal:///./node_modules/@azure/communication-react/dist/dist-esm/react-components/src/components/DevicesButton.js:134
    executeItemClick webpack-internal:///./node_modules/@fluentui/react/lib/components/ContextualMenu/ContextualMenu.base.js:551
    onItemClickBase webpack-internal:///./node_modules/@fluentui/react/lib/components/ContextualMenu/ContextualMenu.base.js:522
    _onItemClick webpack-internal:///./node_modules/@fluentui/react/lib/components/ContextualMenu/ContextualMenuItemWrapper/ContextualMenuItemWrapper.js:26
    React 19
    forEach self-hosted:4288
    React 16

To Reproduce Steps to reproduce the behavior:

  1. Setup call and start or join to room call
  2. Select microphone with .selectMicrophone(newMicrophone)

Expected behavior Microphone to be changed without error

Additional context Tested with Azure calling sample app (https://github.com/Azure-Samples/communication-services-web-calling-hero) and with my own application.

This error does not happen in version 1.23.2.

This error does not happen BEFORE joining the room so user can change microphone without error before joining.

github-actions[bot] commented 1 day ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @acsdevx-msft.