Closed aviramha closed 5 months ago
@aviramha can you please look into the unit tests that are failing? I can reproduce the failures locally, and they are a regression introduced by this PR
Sure! I had an assumption those tests require auth. Fixed now.
Sorry about the spam! This builds upon the other 2 PRs #109 #110 Just out of comfort assuming if merged I'll work on it serially. This adds logic to enable token re-creation when needed, and also making the auth API more fluent. I had a situation where my token got expired mid-upload and there was no place to renew, and if I could break the API I'd have structured it a bit different, but I think is ok.