Skip to content

Commit 4b79ce4

Browse files
common: add x-goog-api-client to all API requests
1 parent f0812b6 commit 4b79ce4

2 files changed

Lines changed: 16 additions & 1 deletion

File tree

packages/common/src/service.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,8 +115,10 @@ Service.prototype.request_ = function(reqOpts, callback) {
115115

116116
delete reqOpts.interceptors_;
117117

118+
var pkg = this.packageJson;
118119
reqOpts.headers = extend({}, reqOpts.headers, {
119-
'User-Agent': util.getUserAgentFromPackageJson(this.packageJson)
120+
'User-Agent': util.getUserAgentFromPackageJson(pkg),
121+
'x-goog-api-client': `gl-node/${process.versions.node} gccl/${pkg.version}`
120122
});
121123

122124
return this.makeAuthenticatedRequest(reqOpts, callback);

packages/common/test/service.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -254,6 +254,19 @@ describe('Service', function() {
254254
service.request_(reqOpts, assert.ifError);
255255
});
256256

257+
it('should add the api-client header', function(done) {
258+
service.makeAuthenticatedRequest = function(reqOpts) {
259+
var pkg = service.packageJson;
260+
assert.strictEqual(
261+
reqOpts.headers['x-goog-api-client'],
262+
`gl-node/${process.versions.node} gccl/${pkg.version}`
263+
);
264+
done();
265+
};
266+
267+
service.request_(reqOpts, assert.ifError);
268+
});
269+
257270
describe('projectIdRequired', function() {
258271
describe('false', function() {
259272
it('should include the projectId', function(done) {

0 commit comments

Comments
 (0)