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

Changing zoom account setup can trigger Heimdall failures #100

Open
Shadowfiend opened this issue Aug 12, 2019 · 2 comments
Open

Changing zoom account setup can trigger Heimdall failures #100

Shadowfiend opened this issue Aug 12, 2019 · 2 comments

Comments

@Shadowfiend
Copy link
Contributor

Relevant flowdock discussion.

In this particular case, a user changed their login email from one address to another, and this started triggering 404s when hubot when to look up the old address's meeting info:

[Mon Aug 12 2019 17:30:55 GMT+0000 (Coordinated Universal Time)] ERROR Failed to fetch next available meeting: { Error: Request failed with status code 404 at createError (/hubot/node_modules/axios/lib/core/createError.js:16:15) at settle (/hubot/node_modules/axios/lib/core/settle.js:17:12) at IncomingMessage.handleStreamEnd (/hubot/node_modules/axios/lib/adapters/http.js:237:11) at IncomingMessage.emit (events.js:187:15) at endReadableNT (_stream_readable.js:1085:12) at process._tickCallback (internal/process/next_tick.js:63:19)

Further down, when looking at the actual 404 response:

User [email protected] not exist or not belong to this account.

@Shadowfiend
Copy link
Contributor Author

Shadowfiend commented Aug 13, 2019

I think our basic fix here is to update nextAvailableMeeting to use Promise.allSettled instead of Promise.all, and, if there are any failed users, refresh the users list and try again.

EDIT: On second thought, rather than refreshing and trying again, use the ones that worked, and also refresh the users list for future attempts?

@kb0rg
Copy link
Contributor

kb0rg commented Nov 6, 2019

There was another occurrence of this here -- in this case some old accounts were pruned but Heimdall didn't get the message.

Note we also realized, during that thread, that we're not making use of the UserType to filter pro accounts. May want to include that update in the fix for this issue. [EDIT: created issue #164 to address the UserType thing]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants