DM Flist : gedis server errors can't be handeled with pyredis

threefoldtecharchive / digital_me

Other

0 stars 2 forks source link

DM Flist : gedis server errors can't be handeled with pyredis #62

Closed siddiquagig closed 5 years ago

siddiquagig commented 6 years ago

The gedis server needs to use aioredis python library instead of pyredis for error handling, but we can't use aioredis for that. We need to fix the error handling using pyredis

zaibon commented 6 years ago

You cannot use aioredis in the 3bot. aioredis is a lib made for asyncio, so it's not compatible with gevent

siddiquagig commented 5 years ago

Error handling is fine now and it shows the error: correct

zaibon commented 5 years ago

@siddiquagig reopening this. From the screen shot I can already 2 problems:

can't we make the error looks a bit nicer ? Now this looks pretty bland
The error message is not something you can show to a end user. code:default:wegateway:list_host is not understandable by anyone

arahmanhamdy commented 5 years ago

The error messages aren't related to the gedis server itself, every chatflow should implement its own messages and exceptions.