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

Triagebot on Zulip: "Failed to query team API: could not get team data" #332

Open
dtolnay opened this issue Feb 8, 2020 · 2 comments
Open
Labels
A-notifications Area: the notifications manager A-zulip Area: Zulip integration bug Something isn't working

Comments

@dtolnay
Copy link
Member

dtolnay commented Feb 8, 2020

I sent a "meta" command to triagebot on Zulip and got back:

Failed to query team API: could not get team data

Caused by:
0: error sending request for url (https://team-api.infra.rust-lang.org/v1/zulip-map.json): connection error: Connection reset by peer (os error 104)
1: connection error: Connection reset by peer (os error 104)
2: Connection reset by peer (os error 104)

I resent the same command ~5 seconds later and it worked. Maybe this needs some retry logic?

@Mark-Simulacrum
Copy link
Member

Hm, interesting. I wonder if this is a stale reqwest client being used -- I recall that the non-async variant could hit this if the connection died when we went to go use it, but I thought that had gotten fixed in the async rewrite.

@Mark-Simulacrum
Copy link
Member

I've added some retry handling code to team API access at least.

@ehuss ehuss added bug Something isn't working A-notifications Area: the notifications manager A-zulip Area: Zulip integration labels Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-notifications Area: the notifications manager A-zulip Area: Zulip integration bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants