Skip to content
This repository has been archived by the owner on Mar 7, 2024. It is now read-only.

Better deepgram error reporting #112

Open
davehorton opened this issue Apr 26, 2023 · 0 comments
Open

Better deepgram error reporting #112

davehorton opened this issue Apr 26, 2023 · 0 comments

Comments

@davehorton
Copy link
Collaborator

If an invalid model/language combination is specified, deepgram rejects the connection request and (probably) provides some error info in the HTTP non-success response, but all mod_deepgram__transcribe logs is that the HS upgrade failed, e.g.

2023-04-26 12:32:00.805091 [DEBUG] dg_transcribe_glue.cpp:265 path: /v1/listen?tier=base&model=conversationalai&language=es&interim_results=true&encoding=linear16&sample_rate=8000
2023-04-26 12:32:00.805091 [DEBUG] dg_transcribe_glue.cpp:306 (1) no resampling needed for this call
2023-04-26 12:32:00.805091 [DEBUG] dg_transcribe_glue.cpp:309 (1) fork_data_init
2023-04-26 12:32:00.805091 [DEBUG] dg_transcribe_glue.cpp:381 connecting now
2023-04-26 12:32:00.805091 [DEBUG] dg_transcribe_glue.cpp:383 connection in progress
2023-04-26 12:32:00.805091 [DEBUG] mod_deepgram_transcribe.c:41 Got SWITCH_ABC_TYPE_INIT.
2023-04-26 12:32:00.805091 [DEBUG] switch_core_media_bug.c:970 Attaching BUG to sofia/drachtio_mrf/nobody@10.12.3.200:5060
2023-04-26 12:32:00.805091 [DEBUG] mod_deepgram_transcribe.c:98 added media bug for dg transcribe
2023-04-26 12:32:00.825081 [DEBUG] switch_core_io.c:448 Setting BUG Codec PCMA:8
2023-04-26 12:32:01.185081 [NOTICE] dg_transcribe_glue.cpp:223 connection failed: HS: ws upgrade response not 101

We should log more details from the http response and if a header or reason is provided send that back to the calling application

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant