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.

Sign up for GitHub

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

Jump to bottom

Handle ApiTelegramException as RasaException #7906

Closed
sentry-io bot opened this issue Feb 8, 2021 · 3 comments · Fixed by #8738
Closed

Handle ApiTelegramException as RasaException #7906

sentry-io bot opened this issue Feb 8, 2021 · 3 comments · Fixed by #8738
Assignees
@ancalita
Labels
area:rasa-oss/channels 📞 Issues focused around rasas input and output channels area:rasa-oss 🎡 Anything related to the open source Rasa framework effort:atom-squad/1 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. tool:sentry type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@sentry-io
Copy link

sentry-io bot commented Feb 8, 2021

Sentry Issue: RASA-OPEN-SOURCE-4XQ

ApiTelegramException: A request to the Telegram API was unsuccessful. Error code: 400. Description: Bad Request: bad webhook: Failed to resolve host: Name or service not known
(9 additional frame(s) were not displayed)
...
  File "rasa/core/run.py", line 187, in serve_application
    conversation_id=conversation_id,
  File "rasa/core/run.py", line 116, in configure_app
    channels.channel.register(input_channels, app, route=route)
  File "rasa/core/channels/channel.py", line 92, in register
    app.blueprint(channel.blueprint(handler), url_prefix=p)
  File "rasa/core/channels/telegram.py", line 187, in blueprint
    out_channel = self.get_output_channel()
  File "rasa/core/channels/telegram.py", line 272, in get_output_channel
    channel.set_webhook(url=self.webhook_url)

This can be caught and a RasaException thrown, as it is caused by incorrect configuration.
@joejuzl joejuzl added area:rasa-oss 🎡 Anything related to the open source Rasa framework area:rasa-oss/channels 📞 Issues focused around rasas input and output channels tool:sentry type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors. labels Feb 8, 2021
@sentry-io
Copy link
Author

sentry-io bot commented Feb 15, 2021

Sentry issue: RASA-OPEN-SOURCE-683

@sentry-io
Copy link
Author

sentry-io bot commented Mar 9, 2021

Sentry issue: RASA-OPEN-SOURCE-6HT

@wochinge
Copy link
Contributor

This issue is causing a lot of sentry events and should hence be prioritized

@TyDunn TyDunn added the effort:atom-squad/1 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. label May 21, 2021
@ancalita ancalita self-assigned this May 24, 2021
@ancalita ancalita mentioned this issue May 24, 2021
Merged
4 tasks
@ancalita ancalita linked a pull request May 24, 2021 that will close this issue
Re-raise ApiTelegramException as RasaException #8738
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ancalita ancalita

Labels
area:rasa-oss/channels 📞 Issues focused around rasas input and output channels area:rasa-oss 🎡 Anything related to the open source Rasa framework effort:atom-squad/1 Label which is used by the Rasa Atom squad to do internal estimation of task sizes. tool:sentry type:bug 🐛 Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Re-raise ApiTelegramException as RasaException RasaHQ/rasa
4 participants
@joejuzl @wochinge @TyDunn @ancalita