Added patched-vscode changes

aws-spenceng · aws-spenceng · commit 1fc7a764599d · 2024-03-07T16:55:28.000-08:00
diff --git a/patched-vscode/build/gulpfile.extensions.js b/patched-vscode/build/gulpfile.extensions.js
@@ -63,7 +63,6 @@ const compilations = [
 	'php-language-features/tsconfig.json',
 	'search-result/tsconfig.json',
 	'references-view/tsconfig.json',
-	'sagemaker-extension/tsconfig.json',
 	'simple-browser/tsconfig.json',
 	'tunnel-forwarding/tsconfig.json',
 	'typescript-language-features/test-workspace/tsconfig.json',
diff --git a/patched-vscode/build/npm/dirs.js b/patched-vscode/build/npm/dirs.js
@@ -39,7 +39,6 @@ const dirs = [
 	'extensions/npm',
 	'extensions/php-language-features',
 	'extensions/references-view',
-	'extensions/sagemaker-extension',
 	'extensions/search-result',
 	'extensions/simple-browser',
 	'extensions/tunnel-forwarding',
diff --git a/patched-vscode/src/vs/base/common/network.ts b/patched-vscode/src/vs/base/common/network.ts
@@ -168,9 +168,7 @@ class RemoteAuthoritiesImpl {
 		return URI.from({
 			scheme: platform.isWeb ? this._preferredWebSchema : Schemas.vscodeRemoteResource,
 			authority: `${host}:${port}`,
-			path: platform.isWeb
-				? (window.location.pathname + "/" + this._remoteResourcesPath).replace(/\/\/+/g, "/")
-				: this._remoteResourcesPath,
+			path: this._remoteResourcesPath,
 			query
 		});
 	}
diff --git a/patched-vscode/src/vs/base/common/product.ts b/patched-vscode/src/vs/base/common/product.ts
@@ -55,7 +55,6 @@ export type ExtensionVirtualWorkspaceSupport = {
 };
 
 export interface IProductConfiguration {
-	readonly rootEndpoint?: string
 	readonly version: string;
 	readonly date?: string;
 	readonly quality?: string;
diff --git a/patched-vscode/src/vs/code/browser/workbench/workbench-dev.html b/patched-vscode/src/vs/code/browser/workbench/workbench-dev.html
@@ -38,7 +38,7 @@
 	<script src="{{WORKBENCH_WEB_BASE_URL}}/out/vs/loader.js"></script>
 	<script src="{{WORKBENCH_WEB_BASE_URL}}/out/vs/webPackagePaths.js"></script>
 	<script>
-		const baseUrl = new URL('{{WORKBENCH_WEB_BASE_URL}}', window.location).toString();
+		const baseUrl = new URL('{{WORKBENCH_WEB_BASE_URL}}', window.location.origin).toString();
 		Object.keys(self.webPackagePaths).map(function (key, index) {
 			self.webPackagePaths[key] = `${baseUrl}/remote/web/node_modules/${key}/${self.webPackagePaths[key]}`;
 		});
diff --git a/patched-vscode/src/vs/code/browser/workbench/workbench.html b/patched-vscode/src/vs/code/browser/workbench/workbench.html
@@ -36,7 +36,7 @@
 	<script src="{{WORKBENCH_WEB_BASE_URL}}/out/vs/loader.js"></script>
 	<script src="{{WORKBENCH_WEB_BASE_URL}}/out/vs/webPackagePaths.js"></script>
 	<script>
-		const baseUrl = new URL('{{WORKBENCH_WEB_BASE_URL}}', window.location).toString();
+		const baseUrl = new URL('{{WORKBENCH_WEB_BASE_URL}}', window.location.origin).toString();
 		Object.keys(self.webPackagePaths).map(function (key, index) {
 			self.webPackagePaths[key] = `${baseUrl}/node_modules/${key}/${self.webPackagePaths[key]}`;
 		});
diff --git a/patched-vscode/src/vs/code/browser/workbench/workbench.ts b/patched-vscode/src/vs/code/browser/workbench/workbench.ts
@@ -304,8 +304,7 @@ class LocalStorageURLCallbackProvider extends Disposable implements IURLCallback
 			this.startListening();
 		}
 
-		const path = (window.location.pathname + "/" + this._callbackRoute).replace(/\/\/+/g, "/");
-		return URI.parse(window.location.href).with({ path: path, query: queryParams.join('&') });
+		return URI.parse(window.location.href).with({ path: this._callbackRoute, query: queryParams.join('&') });
 	}
 
 	private startListening(): void {
diff --git a/patched-vscode/src/vs/platform/extensionResourceLoader/common/extensionResourceLoader.ts b/patched-vscode/src/vs/platform/extensionResourceLoader/common/extensionResourceLoader.ts
@@ -16,6 +16,7 @@ import { getServiceMachineId } from 'vs/platform/externalServices/common/service
 import { IStorageService } from 'vs/platform/storage/common/storage';
 import { TelemetryLevel } from 'vs/platform/telemetry/common/telemetry';
 import { getTelemetryLevel, supportsTelemetry } from 'vs/platform/telemetry/common/telemetryUtils';
+import { RemoteAuthorities } from 'vs/base/common/network';
 import { getRemoteServerRootPath } from 'vs/platform/remote/common/remoteHosts';
 import { TargetPlatform } from 'vs/platform/extensions/common/extensions';
 
@@ -101,7 +102,7 @@ export abstract class AbstractExtensionResourceLoaderService implements IExtensi
 					: version,
 				path: 'extension'
 			}));
-			return this._isWebExtensionResourceEndPoint(uri) ? URI.joinPath(URI.parse(window.location.href), uri.path) : uri;
+			return this._isWebExtensionResourceEndPoint(uri) ? uri.with({ scheme: RemoteAuthorities.getPreferredWebSchema() }) : uri;
 		}
 		return undefined;
 	}
diff --git a/patched-vscode/src/vs/platform/remote/browser/browserSocketFactory.ts b/patched-vscode/src/vs/platform/remote/browser/browserSocketFactory.ts
@@ -280,7 +280,6 @@ export class BrowserSocketFactory implements ISocketFactory<RemoteConnectionType
 	connect({ host, port }: WebSocketRemoteConnection, path: string, query: string, debugLabel: string): Promise<ISocket> {
 		return new Promise<ISocket>((resolve, reject) => {
 			const webSocketSchema = (/^https:/.test(window.location.href) ? 'wss' : 'ws');
-			path = (window.location.pathname + "/" + path).replace(/\/\/+/g, "/")
 			const socket = this._webSocketFactory.create(`${webSocketSchema}://${(/:/.test(host) && !/\[/.test(host)) ? `[${host}]` : host}:${port}${path}?${query}&skipWebSocketFrames=false`, debugLabel);
 			const errorListener = socket.onError(reject);
 			socket.onOpen(() => {
diff --git a/patched-vscode/src/vs/platform/telemetry/common/1dsAppender.ts b/patched-vscode/src/vs/platform/telemetry/common/1dsAppender.ts
@@ -18,8 +18,8 @@ export interface IAppInsightsCore {
 	unload(isAsync: boolean, unloadComplete: (unloadState: ITelemetryUnloadState) => void): void;
 }
 
-const endpointUrl = 'https://0.0.0.0/OneCollector/1.0';
-const endpointHealthUrl = 'https://0.0.0.0/ping';
+const endpointUrl = 'https://mobile.events.data.microsoft.com/OneCollector/1.0';
+const endpointHealthUrl = 'https://mobile.events.data.microsoft.com/ping';
 
 async function getClient(instrumentationKey: string, addInternalFlag?: boolean, xhrOverride?: IXHROverride): Promise<IAppInsightsCore> {
 	const oneDs = await importAMDNodeModule<typeof import('@microsoft/1ds-core-js')>('@microsoft/1ds-core-js', 'dist/ms.core.js');
diff --git a/patched-vscode/src/vs/platform/telemetry/common/telemetryService.ts b/patched-vscode/src/vs/platform/telemetry/common/telemetryService.ts
@@ -204,15 +204,15 @@ Registry.as<IConfigurationRegistry>(Extensions.Configuration).registerConfigurat
 	'properties': {
 		[TELEMETRY_SETTING_ID]: {
 			'type': 'string',
-			'enum': [TelemetryConfiguration.OFF],
+			'enum': [TelemetryConfiguration.ON, TelemetryConfiguration.ERROR, TelemetryConfiguration.CRASH, TelemetryConfiguration.OFF],
 			'enumDescriptions': [
 				localize('telemetry.telemetryLevel.default', "Sends usage data, errors, and crash reports."),
 				localize('telemetry.telemetryLevel.error', "Sends general error telemetry and crash reports."),
 				localize('telemetry.telemetryLevel.crash', "Sends OS level crash reports."),
 				localize('telemetry.telemetryLevel.off', "Disables all product telemetry.")
 			],
 			'markdownDescription': getTelemetryLevelSettingDescription(),
-			'default': TelemetryConfiguration.OFF,
+			'default': TelemetryConfiguration.ON,
 			'restricted': true,
 			'scope': ConfigurationScope.APPLICATION,
 			'tags': ['usesOnlineServices', 'telemetry']
diff --git a/patched-vscode/src/vs/platform/update/common/update.config.contribution.ts b/patched-vscode/src/vs/platform/update/common/update.config.contribution.ts
@@ -18,7 +18,7 @@ configurationRegistry.registerConfiguration({
 		'update.mode': {
 			type: 'string',
 			enum: ['none', 'manual', 'start', 'default'],
-			default: 'none',
+			default: 'default',
 			scope: ConfigurationScope.APPLICATION,
 			description: localize('updateMode', "Configure whether you receive automatic updates. Requires a restart after change. The updates are fetched from a Microsoft online service."),
 			tags: ['usesOnlineServices'],
@@ -50,7 +50,7 @@ configurationRegistry.registerConfiguration({
 		},
 		'update.showReleaseNotes': {
 			type: 'boolean',
-			default: false,
+			default: true,
 			scope: ConfigurationScope.APPLICATION,
 			description: localize('showReleaseNotes', "Show Release Notes after an update. The Release Notes are fetched from a Microsoft online service."),
 			tags: ['usesOnlineServices']
diff --git a/patched-vscode/src/vs/server/node/serverEnvironmentService.ts b/patched-vscode/src/vs/server/node/serverEnvironmentService.ts
@@ -87,9 +87,6 @@ export const serverOptions: OptionDescriptions<Required<ServerParsedArgs>> = {
 
 	'compatibility': { type: 'string' },
 
-	/* ----- sagemaker ----- */
-	'base-path': { type: 'string' },
-
 	_: OPTIONS['_']
 };
 
@@ -206,9 +203,6 @@ export interface ServerParsedArgs {
 
 	compatibility: string;
 
-	/* ----- sagemaker ----- */
-	'base-path'?: string,
-
 	_: string[];
 }
 
diff --git a/patched-vscode/src/vs/server/node/webClientServer.ts b/patched-vscode/src/vs/server/node/webClientServer.ts
@@ -278,7 +278,7 @@ export class WebClientServer {
 		const remoteAuthority = (
 			useTestResolver
 				? 'test+test'
-				: (getFirstHeader('x-original-host') || getFirstHeader('x-forwarded-host') || req.headers.host || window.location.host)
+				: (getFirstHeader('x-original-host') || getFirstHeader('x-forwarded-host') || req.headers.host)
 		);
 		if (!remoteAuthority) {
 			return serveError(req, res, 400, `Bad request.`);
@@ -305,12 +305,7 @@ export class WebClientServer {
 			scopes: [['user:email'], ['repo']]
 		} : undefined;
 
-		const basePath: string = this._environmentService.args['base-path'] || "/"
-		const base = relativeRoot(basePath)
-		const vscodeBase = relativePath(basePath)
-
 		const productConfiguration = <Partial<IProductConfiguration>>{
-			rootEndpoint: base,
 			embedderIdentifier: 'server-distro',
 			extensionsGallery: this._webExtensionResourceUrlTemplate ? {
 				...this._productService.extensionsGallery,
@@ -345,10 +340,8 @@ export class WebClientServer {
 		const values: { [key: string]: string } = {
 			WORKBENCH_WEB_CONFIGURATION: asJSON(workbenchWebConfiguration),
 			WORKBENCH_AUTH_SESSION: authSessionInfo ? asJSON(authSessionInfo) : '',
-			WORKBENCH_WEB_BASE_URL: vscodeBase + this._staticRoute,
-			WORKBENCH_NLS_BASE_URL: vscodeBase + (nlsBaseUrl ? `${nlsBaseUrl}${!nlsBaseUrl.endsWith('/') ? '/' : ''}${this._productService.commit}/${this._productService.version}/` : ''),
-			BASE: base,
-			VS_BASE: vscodeBase,
+			WORKBENCH_WEB_BASE_URL: this._staticRoute,
+			WORKBENCH_NLS_BASE_URL: nlsBaseUrl ? `${nlsBaseUrl}${!nlsBaseUrl.endsWith('/') ? '/' : ''}${this._productService.commit}/${this._productService.version}/` : '',
 		};
 
 		if (useTestResolver) {
@@ -446,60 +439,3 @@ export class WebClientServer {
 		return void res.end(data);
 	}
 }
-
-
-/**
- * Remove extra slashes in a URL.
- *
- * This is meant to fill the job of `path.join` so you can concatenate paths and
- * then normalize out any extra slashes.
- *
- * If you are using `path.join` you do not need this but note that `path` is for
- * file system paths, not URLs.
- */
-export const normalizeUrlPath = (url: string, keepTrailing = false): string => {
-	return url.replace(/\/\/+/g, "/").replace(/\/+$/, keepTrailing ? "/" : "")
-}
-
-/**
- * Get the relative path that will get us to the root of the page. For each
- * slash we need to go up a directory.  Will not have a trailing slash.
- *
- * For example:
- *
- * / => .
- * /foo => .
- * /foo/ => ./..
- * /foo/bar => ./..
- * /foo/bar/ => ./../..
- *
- * All paths must be relative in order to work behind a reverse proxy since we
- * we do not know the base path.  Anything that needs to be absolute (for
- * example cookies) must get the base path from the frontend.
- *
- * All relative paths must be prefixed with the relative root to ensure they
- * work no matter the depth at which they happen to appear.
- *
- * For Express `req.originalUrl` should be used as they remove the base from the
- * standard `url` property making it impossible to get the true depth.
- */
-export const relativeRoot = (originalUrl: string): string => {
-	const depth = (originalUrl.split("?", 1)[0].match(/\//g) || []).length
-	return normalizeUrlPath("./" + (depth > 1 ? "../".repeat(depth - 1) : ""))
-}
-
-/**
- * Get the relative path to the current resource.
- *
- * For example:
- *
- * / => .
- * /foo => ./foo
- * /foo/ => .
- * /foo/bar => ./bar
- * /foo/bar/ => .
- */
-export const relativePath = (originalUrl: string): string => {
-	const parts = originalUrl.split("?", 1)[0].split("/")
-	return normalizeUrlPath("./" + parts[parts.length - 1])
-}
diff --git a/patched-vscode/src/vs/workbench/contrib/preferences/common/preferencesContribution.ts b/patched-vscode/src/vs/workbench/contrib/preferences/common/preferencesContribution.ts
@@ -144,7 +144,7 @@ registry.registerConfiguration({
 		'workbench.settings.enableNaturalLanguageSearch': {
 			'type': 'boolean',
 			'description': nls.localize('enableNaturalLanguageSettingsSearch', "Controls whether to enable the natural language search mode for settings. The natural language search is provided by a Microsoft online service."),
-			'default': false,
+			'default': true,
 			'scope': ConfigurationScope.WINDOW,
 			'tags': ['usesOnlineServices']
 		},
diff --git a/patched-vscode/src/vs/workbench/electron-sandbox/desktop.contribution.ts b/patched-vscode/src/vs/workbench/electron-sandbox/desktop.contribution.ts
@@ -263,7 +263,7 @@ import { applicationConfigurationNodeBase } from 'vs/workbench/common/configurat
 			'telemetry.enableCrashReporter': {
 				'type': 'boolean',
 				'description': localize('telemetry.enableCrashReporting', "Enable crash reports to be collected. This helps us improve stability. \nThis option requires restart to take effect."),
-				'default': false,
+				'default': true,
 				'tags': ['usesOnlineServices', 'telemetry'],
 				'markdownDeprecationMessage': localize('enableCrashReporterDeprecated', "If this setting is false, no telemetry will be sent regardless of the new setting's value. Deprecated due to being combined into the {0} setting.", `\`#${TELEMETRY_SETTING_ID}#\``),
 			}
diff --git a/patched-vscode/src/vs/workbench/services/assignment/common/assignmentService.ts b/patched-vscode/src/vs/workbench/services/assignment/common/assignmentService.ts
@@ -143,7 +143,7 @@ registry.registerConfiguration({
 		'workbench.enableExperiments': {
 			'type': 'boolean',
 			'description': localize('workbench.enableExperiments', "Fetches experiments to run from a Microsoft online service."),
-			'default': false,
+			'default': true,
 			'scope': ConfigurationScope.APPLICATION,
 			'restricted': true,
 			'tags': ['usesOnlineServices']

Original file line number	Diff line number	Diff line change
`@@ -304,8 +304,7 @@ class LocalStorageURLCallbackProvider extends Disposable implements IURLCallback`
`304`	`304`	`this.startListening();`
`305`	`305`	`}`
`306`	`306`
`307`		`- const path = (window.location.pathname + "/" + this._callbackRoute).replace(/\/\/+/g, "/");`
`308`		`- return URI.parse(window.location.href).with({ path: path, query: queryParams.join('&') });`
	`307`	`+ return URI.parse(window.location.href).with({ path: this._callbackRoute, query: queryParams.join('&') });`
`309`	`308`	`}`
`310`	`309`
`311`	`310`	`private startListening(): void {`
Original file line number	Diff line number	Diff line change
`@@ -18,8 +18,8 @@ export interface IAppInsightsCore {`
`18`	`18`	`unload(isAsync: boolean, unloadComplete: (unloadState: ITelemetryUnloadState) => void): void;`
`19`	`19`	`}`
`20`	`20`
`21`		`-const endpointUrl = 'https://0.0.0.0/OneCollector/1.0';`
`22`		`-const endpointHealthUrl = 'https://0.0.0.0/ping';`
	`21`	`+const endpointUrl = 'https://mobile.events.data.microsoft.com/OneCollector/1.0';`
	`22`	`+const endpointHealthUrl = 'https://mobile.events.data.microsoft.com/ping';`
`23`	`23`
`24`	`24`	`async function getClient(instrumentationKey: string, addInternalFlag?: boolean, xhrOverride?: IXHROverride): Promise<IAppInsightsCore> {`
`25`	`25`	`const oneDs = await importAMDNodeModule<typeof import('@microsoft/1ds-core-js')>('@microsoft/1ds-core-js', 'dist/ms.core.js');`