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

[RSS] Bot -> HS error in the Integration Manager's page #908

Open
opk12 opened this issue Mar 4, 2024 · 1 comment
Open

[RSS] Bot -> HS error in the Integration Manager's page #908

opk12 opened this issue Mar 4, 2024 · 1 comment
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.

Comments

@opk12
Copy link

opk12 commented Mar 4, 2024

Not pushing, just throwing an idea up in the air.

If the bot knows that there was a Bot <-> HS error (as it seems from some comments of #844), should the error appear in the Integrations Manager dialog?

Maybe having an error under Recent feed results, in addition to Successful fetch, would help discover and report any failures in the future.

I don't know if it is feasible? The dialog is loaded from integrations.ems.host, not the HS, so a failure to send to the HS can be logged somehow?

@Half-Shot Half-Shot added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements. label Mar 6, 2024
@Half-Shot
Copy link
Contributor

The problem is in theory even Bot <-> HS errors should be retried, and they should be invisible ideally. I'm worried about bubbling up operational problems like homeserver flakiness to the user, whereas the feed fetch failures are more understandable to the user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements.
Projects
None yet
Development

No branches or pull requests

2 participants