Antonio Lobato

Results 199 comments of Antonio Lobato

This CL is a WIP -- while all current tests pass and the changes are backwards compatible, there are no tests for the new functionality yet.

Okay, it turns out the base auth methods are tested because I moved around some of the token verification code to use the new auth options. PTAL!

Hi folks, Sorry, I've been taking time off for the holidays here in the US. I'll finish this up next week when I return. @bweston92 Currently, the API is backwards...

I'm absolutely supportive of that change @bweston92 -- you're right that it was added quickly. I wanted to make minimal changes, and at the time it seemed like the easiest...

Sorry, one more thing @bweston92; my only concern with your outlined approach is that it's done correctly in the context of this client. If we're going to implement the whole...

Absolutely! So what I've been working on is a decoupled system in which outbound Twitch API calls are processed in their own stand alone microservice via RPC's. Right now, if...

I'll take a look and outline what changes would need to be done in order to get this in place. At the very least, we should be able to measure...

What I ended up doing as a work around is instantiating a second client just for calls that need an AppAccessToken and making sure to never set a UserAccessToken on...

Reasonable response, I'll go ahead and send one over soon.