I noticed that while making batch requests, 401 Unauthorized responses are not caught in the current handler because they return an HTTP status code of 200, and each error is presented in a JSON array in the response body.
I managed to temporarily fix this by changing the OAuth2Client.prototype.request function:
from:
var hasAuthError = (res.statusCode == 401 || res.statusCode == 403)
to:
// TODO: Check if it's not userRateLimitExceeded
var hasAuthError =
(res.statusCode == 401 || res.statusCode == 403)
|| body.indexOf('401 Unauthorized') >= 0; // batch requests
A deep copy of the request options must be cached:
from:
var that = this;
var credentials = this.credentials;
to:
var optsClone = JSON.parse(JSON.stringify(opts));
var that = this;
var credentials = this.credentials;
and used for the request when it is re-played
from:
var tokens = result;
tokens.refresh_token = credentials.refresh_token;
that.credentials = tokens;
that.request(opts, opt_callback, true);
to:
var tokens = result;
tokens.refresh_token = credentials.refresh_token;
that.credentials = tokens;
that.request(optsClone, opt_callback, true);
I'm sure there is probably a more elegant way of fixing this instead of using "indexOf", but this should help people having the same problem for now.
I noticed that while making batch requests, 401 Unauthorized responses are not caught in the current handler because they return an HTTP status code of 200, and each error is presented in a JSON array in the response body.
I managed to temporarily fix this by changing the OAuth2Client.prototype.request function:
from:
to:
A deep copy of the request options must be cached:
from:
to:
and used for the request when it is re-played
from:
to:
I'm sure there is probably a more elegant way of fixing this instead of using "indexOf", but this should help people having the same problem for now.