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

[IcM] Speech via Cognitive Services does not work on Cordova on Android #3281

Closed
1 task done
compulim opened this issue Jun 26, 2020 · 3 comments · Fixed by #3508
Closed
1 task done

[IcM] Speech via Cognitive Services does not work on Cordova on Android #3281

compulim opened this issue Jun 26, 2020 · 3 comments · Fixed by #3508
Assignees
Labels
area-docs Documentation required Bot Services Required for internal Azure reporting. Do not delete. Do not change color. bug Indicates an unexpected problem or an unintended behavior. customer-replied-to Required for internal reporting. Do not delete. customer-reported Required for internal Azure reporting. Do not delete. icm Linked from IcM p1 Painful if we don't fix, won't block releasing
Milestone

Comments

@compulim
Copy link
Contributor

compulim commented Jun 26, 2020

Screenshots

(TBD)

Version

4.9.2

Describe the bug

Customer reported Web Chat hosted on a Cordova app on Android with speech enabled through Cognitive Services does not work.

From the support ticket:

Customer has integrated bot into angular application and text-speech and speech-text functionality works fine. He has converted the same application into Cordova app where clicking on mic, it doesn’t work. Microphone doesn’t even listen to the audio.

It seems speech recognition is working but not speech synthesis, need to investigate.

Steps to reproduce

From the support ticket:

  • Set up android studio and collected HAR trace using fiddler as proxy.
  • We have the angular source code and the .apk file.
  • Created new bot and speech service and replaced the secrets in Customer’s code.
  • I have also made changes to the code so that it requires permission to Microphone and provided access to microphone. Still it fails.
  • In fiddler, I see that to Speech service, after GET /speech/recognition/conversation/cognitiveservices/v1, I don’t see any request to POST /cognitiveservices/v1. Is it a problem because the Microphone not recording the audio input?

Expected behavior

Speech via Cognitive Services should work on Cordova on Android. If not working, we need to understand why and document it in docs/SPEECH.md

Next steps

  • Repro the issue with a vanilla Cordova app with appropriate app permissions

Additional context

The ticket number is 194036845/120060926003891.

[Bug]

@compulim compulim added bug Indicates an unexpected problem or an unintended behavior. customer-reported Required for internal Azure reporting. Do not delete. Bot Services Required for internal Azure reporting. Do not delete. Do not change color. customer-replied-to Required for internal reporting. Do not delete. R10 labels Jun 26, 2020
@compulim compulim self-assigned this Jun 26, 2020
@compulim compulim changed the title Speech via Cognitive Services does not work on Cordova on Android [CSS] Speech via Cognitive Services does not work on Cordova on Android Jun 26, 2020
@compulim compulim changed the title [CSS] Speech via Cognitive Services does not work on Cordova on Android [IcM] Speech via Cognitive Services does not work on Cordova on Android Jun 26, 2020
@compulim compulim added the icm Linked from IcM label Jun 26, 2020
@cwhitten cwhitten added the p1 Painful if we don't fix, won't block releasing label Aug 7, 2020
@compulim
Copy link
Contributor Author

compulim commented Aug 7, 2020

Tested with Cordova 10 on an Android 10 device.

A permission dialog is required to enable speech support inside Cordova app.

I tested this Cordova plug-in cordova-plugin-k-webrtc-permission and it can correctly request permission and enable speech support in Web Chat.

The plug-in request for permissions more than Web Chat is needed, so I would suggested looking for another approach similar to what this plug-in does.

@compulim
Copy link
Contributor Author

compulim commented Aug 7, 2020

We need to add more documentation before closing this issue.

@compulim compulim added R11 and removed R10 labels Aug 7, 2020
@compulim
Copy link
Contributor Author

compulim commented Aug 7, 2020

Moving to R11 for documentation update.

@compulim compulim added the area-docs Documentation required label Aug 7, 2020
@compulim compulim added this to the R11 milestone Aug 10, 2020
@cwhitten cwhitten assigned corinagum and unassigned compulim Sep 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-docs Documentation required Bot Services Required for internal Azure reporting. Do not delete. Do not change color. bug Indicates an unexpected problem or an unintended behavior. customer-replied-to Required for internal reporting. Do not delete. customer-reported Required for internal Azure reporting. Do not delete. icm Linked from IcM p1 Painful if we don't fix, won't block releasing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants