Skip to content

Commit 0148141

Browse files
committed
Avoid using buitlin URL because it is flagged by the layers checker
1 parent 5ecafcc commit 0148141

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/vs/server/node/remoteExtensionHostAgentServer.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,7 @@ export class RemoteExtensionHostAgentServer extends Disposable implements IServe
149149
responseHeaders['Vary'] = 'Origin';
150150
const requestOrigin = req.headers['origin'];
151151
if (requestOrigin && this._webEndpointOriginChecker.matches(requestOrigin)) {
152+
console.log(`setting Access-Control-Allow-Origin`);
152153
responseHeaders['Access-Control-Allow-Origin'] = requestOrigin;
153154
}
154155

@@ -789,13 +790,13 @@ class WebEndpointOriginChecker {
789790
}
790791

791792
const uuid = generateUuid();
792-
const exampleUrl = new URL(
793+
const exampleUri = URI.parse(
793794
webEndpointUrlTemplate
794795
.replace('{{uuid}}', uuid)
795796
.replace('{{commit}}', commit)
796797
.replace('{{quality}}', quality)
797798
);
798-
const exampleOrigin = exampleUrl.origin;
799+
const exampleOrigin = `${exampleUri.scheme}://${exampleUri.authority}`;
799800
const originRegExpSource = (
800801
escapeRegExpCharacters(exampleOrigin)
801802
.replace(uuid, '[a-zA-Z0-9\-]+')

0 commit comments

Comments
 (0)