minds-ai / zoom-drive-connector

Automatically uploads Zoom meeting recordings to Google Drive.
Other
33 stars 8 forks source link

Some minor bugfixes #6

Closed jbedorf closed 5 years ago

jbedorf commented 5 years ago

Saw you added an issue, so wanted to make sure these fixes are included before you continue. I am not 100% sure that the httplib fixes all broken_pipe errors since those things happen at random. There is one more fix that I have to look into/make and that is an occasional connection error that should be caught instead of crashing the whole program.

MrFlynn commented 5 years ago

Is this in reference to issue #4?

MrFlynn commented 5 years ago

Regardless, it looks like your fix is the recommended solution in this thread. I'll approve the changes since that seems like the correct fix.

jbedorf commented 5 years ago

4 is slightly different, I still have to dive deeper into that and got some more stack traces to help me narrow that down. It just happens so irregular its hard to test.

I indeed took the fix from that thread :)