Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Only legacy OSS protocol dialect is supported #855

Closed
RouR opened this issue Jan 3, 2020 · 9 comments
Closed

Only legacy OSS protocol dialect is supported #855

RouR opened this issue Jan 3, 2020 · 9 comments

Comments

@RouR
Copy link

RouR commented Jan 3, 2020

I have issue with library Selenium.WebDriver 4.0.0-alpha03

This lib is not campatible with selenoid 1.9.3 after removing legacy OSS protocol dialect - check this commit SeleniumHQ/selenium@8767cd3#diff-5fedfeb3060c9b589bc5ac90334761caL1073

This appears as error

{"status":13,"value":{"message":"Requested environment is not available"}}

@vania-pooh
Copy link
Member

@RouR that means you are specifying capabilities incorrectly. Could you please show your capabilities?

@RouR
Copy link
Author

RouR commented Jan 4, 2020

var browser = new ChromeOptions();
browser.BrowserVersion = "79";
var driver = new RemoteWebDriver(options.HubUrl, browser.ToCapabilities(), options.BrowserTimeout);

This code is ok with Selenium.WebDriver 3.141.0

@vania-pooh
Copy link
Member

@RouR Selenoid supports w3c-style protocol. Anyway, give me some time to check (will do with Java client of the same version, have no C#).

@vania-pooh
Copy link
Member

@RouR for me just the same version of Java client works like a charm. Please try to print request and response body:

CommandExecutor commandExecutor = new HttpCommandExecutor(...);
WebDriver driver = new RemoteWebDriver(commandExecutor, browser);

I'm not a C# developer, so there should be the way to make .NET standard library print JSON request and response body. So far as I can see in this class Selenium is using standard System.Net.Http package to make HTTP requests.

@RouR
Copy link
Author

RouR commented Jan 9, 2020

This work:

{"desiredCapabilities":{"browserName":"chrome","browserVersion":"79","goog:chromeOptions":{"args":[]}},"capabilities":{"firstMatch":[{"browserName":"chrome","browserVersion":"79","goog:chromeOptions":{"args":["--enable-dom-distiller","--distillability-dev"],"localState":{"reader.parse-on-load.enabled":true}}}]}}

{"value":{"capabilities":{"acceptInsecureCerts":false,"browserName":"chrome","browserVersion":"79.0.3945.79","chrome":{"chromedriverVersion":"79.0.3945.36 (3582db32b33893869b8c1339e8f4d9ed1816f143-refs/branch-heads/3945@{#614})","userDataDir":"/tmp/.com.google.Chrome.WzrN4V"},"goog:chromeOptions":{"debuggerAddress":"localhost:40617"},"networkConnectionEnabled":false,"pageLoadStrategy":"normal","platformName":"linux","proxy":{},"setWindowRect":true,"strictFileInteractability":false,"timeouts":{"implicit":0,"pageLoad":300000,"script":30000},"unhandledPromptBehavior":"dismiss and notify"},"sessionId":"f8d46813f4b203e76f6cfea31ec7cd37"}}

This not work:

{"capabilities":{"firstMatch":[{"browserName":"chrome","browserVersion":"79","goog:chromeOptions":{"args":[]}}]}}
{"status":13,"value":{"message":"Requested environment is not available"}}

See missed desiredCapabilities

@vania-pooh
Copy link
Member

@RouR ok, then this is because of missing firstMatch support in Selenoid. Closing this as a duplicate of #450.

@vania-pooh
Copy link
Member

@RouR yesterday we merged support for firstMatch. Could you try whether it works with aerokube/selenoid:latest image?

@RouR
Copy link
Author

RouR commented Jan 14, 2020

Yes, it work!
Thanks a lot

@vania-pooh
Copy link
Member

@RouR ok, this will be released during next days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants