yujunjun/playwright
1
0
Fork 0
mirror of https://github.com/microsoft/playwright.git synced 2025-06-26 21:40:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

Revert "chore: validate client/server version match for remote connections (#10542)" (#10766)

Browse Source 
This reverts commit 41070a2f555a12bda6ac2589ca72cf0d886f989c.
This commit is contained in:
Max Schmitt 2021-12-07 14:45:05 -08:00 committed by GitHub
parent 5d10fbd3d8
commit 300a7f9b44
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
5 changed files with 1 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
packages/playwright-core/src/client/connection.ts
View File

@ -40,7 +40,6 @@ import { Artifact } from './artifact';
import { EventEmitter } from 'events';
import { JsonPipe } from './jsonPipe';
import { APIRequestContext } from './fetch';
import { getPlaywrightVersion } from '../utils/utils';
class Root extends ChannelOwner<channels.RootChannel> {
constructor(connection: Connection) {
@ -50,7 +49,6 @@ class Root extends ChannelOwner<channels.RootChannel> {
async initialize(): Promise<Playwright> {
return Playwright.from((await this._channel.initialize({
sdkLanguage: 'javascript',
version: getPlaywrightVersion(),
})).playwright);
}
}

10
packages/playwright-core/src/dispatchers/dispatcher.ts
View File

@ -18,7 +18,7 @@ import { EventEmitter } from 'events';
import * as channels from '../protocol/channels';
import { serializeError } from '../protocol/serializers';
import { createScheme, Validator, ValidationError } from '../protocol/validator';
import { assert, debugAssert, getPlaywrightVersion, isUnderTest, monotonicTime } from '../utils/utils';
import { assert, debugAssert, isUnderTest, monotonicTime } from '../utils/utils';
import { tOptional } from '../protocol/validatorPrimitives';
import { kBrowserOrContextClosedError } from '../utils/errors';
import { CallMetadata, SdkObject } from '../server/instrumentation';
@ -132,7 +132,6 @@ export class Root extends Dispatcher<{ guid: '' }, any> {
async initialize(params: channels.RootInitializeParams): Promise<channels.RootInitializeResult> {
assert(this.createPlaywright);
assert(!this._initialized);
assertPlaywrightVersion(params.version);
this._initialized = true;
return {
playwright: await this.createPlaywright(this, params),
@ -321,10 +320,3 @@ function formatLogRecording(log: string[]): string {
}
let lastEventId = 0;
function assertPlaywrightVersion(givenVersion: string): void {
const expectedVersion = getPlaywrightVersion();
const givenParts = givenVersion.split('.');
const expectedParts = givenVersion.split('.');
assert(givenParts.slice(0, 2).join('.') === expectedParts.slice(0, 2).join('.'), `${expectedVersion} does not match the client version ${givenVersion}.`);
}

1
packages/playwright-core/src/protocol/channels.ts
View File

@ -356,7 +356,6 @@ export interface RootChannel extends RootEventTarget, Channel {
}
export type RootInitializeParams = {
sdkLanguage: string,
version: string,
};
export type RootInitializeOptions = {

1
packages/playwright-core/src/protocol/protocol.yml
View File

@ -423,7 +423,6 @@ Root:
initialize:
parameters:
sdkLanguage: string
version: string
returns:
playwright: Playwright

1
packages/playwright-core/src/protocol/validator.ts
View File

@ -191,7 +191,6 @@ export function createScheme(tChannel: (name: string) => Validator): Scheme {
scheme.LifecycleEvent = tEnum(['load', 'domcontentloaded', 'networkidle', 'commit']);
scheme.RootInitializeParams = tObject({
sdkLanguage: tString,
version: tString,
});
scheme.PlaywrightSocksConnectedParams = tObject({
uid: tString,