<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">One thing that I would like to see: more attributes on error responses,<br>mainly an error code that can be interpreted by the dialplan in order to<br>react differently, depending of the error type.<br></blockquote><div><br></div><div>Thanks for the feedback!</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">My preference goes to the "error_details" as it is more flexible, but I<br>don't know how complex it is to implement.<br></blockquote><div><br></div><div>This shouldn't be too difficult to do, we would just need a way to query the JSON via dialplan. Maybe something similar to channel variables. Definitely something to think about moving forward.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 29, 2021 at 4:40 PM <a href="mailto:sduthil@wazo.io">sduthil@wazo.io</a> <<a href="mailto:sduthil@wazo.io">sduthil@wazo.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 3/22/21 3:14 PM, Ben Ford wrote:<br>
> More information on the protocol used for this and the overall <br>
> project can be found here:<br>
> <br>
> <a href="https://wiki.asterisk.org/wiki/pages/viewpage.action?pageId=45482453" rel="noreferrer" target="_blank">https://wiki.asterisk.org/wiki/pages/viewpage.action?pageId=45482453</a><br>
> <br>
> After reading the wiki page, if there is anything you feel could be <br>
> improved, we’d love to hear about it.<br>
<br>
Thank you Ben.<br>
<br>
One thing that I would like to see: more attributes on error responses, <br>
mainly an error code that can be interpreted by the dialplan in order to <br>
react differently, depending of the error type. For example, I want to <br>
distinguish when the application received no sound (e.g. because of a <br>
microphone issue) I want the user to know that and try again vs the <br>
application is misconfigured and I just want to stop.<br>
<br>
Using only one "error_msg" attribute for that is not ideal, since the <br>
calling dialplan has to parse the error_msg and react accordingly. <br>
Having a separate error code allows the dialplan to choose the <br>
appropriate behavior easily, while still having the error_msg for <br>
logging/displaying/debugging purposes.<br>
<br>
I see two choices for this improvement:<br>
<br>
- add an "error_code" (or similar) in the error response<br>
<br>
Example:<br>
{<br>
     "version": "1.0",<br>
     "response": "error",<br>
     "error_msg": "Could not connect to Google (server down).",<br>
     "error_code": "10001"  // or any other string from the application<br>
}<br>
<br>
<br>
- allow external applications to add arbitrary attributes into a <br>
"error_details" attribute<br>
<br>
Example:<br>
{<br>
     "version": "1.0",<br>
     "response": "error",<br>
     "error_msg": "Could not connect to Google (server down).",<br>
     "error_details": {<br>
         "error_code": "10001",<br>
         "error_user_language": "fr_CA"<br>
         "error_user_display": "La connexion au serveur Google a échoué"<br>
     }<br>
}<br>
<br>
My preference goes to the "error_details" as it is more flexible, but I <br>
don't know how complex it is to implement.<br>
<br>
PS: I am not currently a user of TTS/STT, I don't have a real use-case <br>
needing this.<br>
<br>
-- <br>
Sébastien Duthil<br>
Wazo developer<br>
<br>
-- <br>
_____________________________________________________________________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" rel="noreferrer" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
   <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" rel="noreferrer" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-dev</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr" style="font-size:12.8px"><span style="background-color:rgb(255,255,255)"><font color="#073763"><font face="tahoma, sans-serif">Benjamin Ford</font><br></font></span></div><div style="font-size:12.8px"><font face="tahoma, sans-serif" style="background-color:rgb(255,255,255)" color="#073763">Software Engineer<br></font></div><div style="font-size:12.8px"><font face="tahoma, sans-serif" style="background-color:rgb(255,255,255)" color="#073763">256-428-6147</font></div><div><font face="tahoma, sans-serif" style="font-size:12.8px"><span style="background-color:rgb(255,255,255)"><font color="#073763">Check us out at</font></span> <a href="http://www.sangoma.com" target="_blank">www.sangoma.com</a> and <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a></font></div><div dir="ltr" style="font-size:12.8px"><div style="font-size:12.8px"><br></div></div></div></div></div></div></div>