add typings installer user agent for npm requests (microsoft#12446) (microsoft#12449)

vladima · web-flow · commit 432f7a9a7d8b · 2016-11-22T15:22:50.000-08:00
* add typings installer user agent for npm requests

* address PR feedback: change name of user agent
diff --git a/src/server/typingsInstaller/nodeTypingsInstaller.ts b/src/server/typingsInstaller/nodeTypingsInstaller.ts
@@ -127,7 +127,7 @@ namespace ts.server.typingsInstaller {
             if (this.log.isEnabled()) {
                 this.log.writeLine(`#${requestId} with arguments'${JSON.stringify(args)}'.`);
             }
-            const command = `${this.npmPath} install ${args.join(" ")} --save-dev`;
+            const command = `${this.npmPath} install ${args.join(" ")} --save-dev --user-agent="typesInstaller/${version}"`;
             const start = Date.now();
             let stdout: Buffer;
             let stderr: Buffer;

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ namespace ts.server.typingsInstaller {`
`127`	`127`	`if (this.log.isEnabled()) {`
`128`	`128`	this.log.writeLine(`#${requestId} with arguments'${JSON.stringify(args)}'.`);
`129`	`129`	`}`
`130`		- const command = `${this.npmPath} install ${args.join(" ")} --save-dev`;
	`130`	+ const command = `${this.npmPath} install ${args.join(" ")} --save-dev --user-agent="typesInstaller/${version}"`;
`131`	`131`	`const start = Date.now();`
`132`	`132`	`let stdout: Buffer;`
`133`	`133`	`let stderr: Buffer;`