Skip to content

Commit

Permalink
fix(client) disable API health-checks
Browse files Browse the repository at this point in the history
Semi-resurrection of #1054

I got logs of a call where this happened again and it doesn't make any
sense.

This PR disables the action but leaves the check in, so we can still see
it in the logs when we dig into issues.
  • Loading branch information
saghul committed Sep 11, 2024
1 parent 96e3309 commit 656c95b
Showing 1 changed file with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -143,10 +143,11 @@ export default class AbstractMeetingFrame extends React.Component {
_onApiHealthCheckError(reason) {
logger.error('api health check failed', { reason });

this._onMeetingLeave({
errorCode: reason,
error: 'appEvents.meetingConnectionLost'
});
// TODO: monitor logs and figure out why we get this failure when things are ok (well, are they?!).
// this._onMeetingLeave({
// errorCode: reason,
// error: 'appEvents.meetingConnectionLost'
// });
}

/**
Expand Down

0 comments on commit 656c95b

Please sign in to comment.