[🐛 Bug]: [dotnet] Adding additional option is ignored

[nvborisenko]

Description

I use AddAdditionalOption() method, and seems it doesn't work.

Reproducible Code

var options = new ChromeOptions();

options.AddAdditionalOption("myCustomOption", "myCastomValue");

using var driver = new ChromeDriver(options);

Debugging Logs

18:03:07.632 TRACE HttpCommandExecutor: >> POST http://localhost:52206/session
{"capabilities":{"firstMatch":[{"browserName":"chrome","goog:chromeOptions":{"binary":"C:\\Program Files\\Google\\Chrome\\Application\\chrome.exe"}}]}}

[selenium-ci]

@nvborisenko, thank you for creating this issue. We will troubleshoot it as soon as we can.

Selenium Triage Team: remember to follow the Triage Guide

[diemol]

You can only add options that are W3C compliant.

[nvborisenko]

Hey, I can add any option. Please see the code above.

[diemol]

Yes, you can. But only W3C compliant are sent in the payload.

[nvborisenko]

Meaning that I should not be able to add non-compliant options? And this issue is about it. Why I use some API, but in reality it doesn't do thing it supposed to do?

[nvborisenko]

you closed this issue as done, in my opinion action items would be:

• improve docs
• throw an error when non-compliant option is provided
• add a message in logs saying "hey man, you are dummy user, selenium doesn't support your case"

[diemol]

https://www.selenium.dev/blog/2022/legacy-protocol-support/

Feel free to improve .NET regarding bullet 2 and 3.

[nvborisenko]

Comic situation.

options.AddAdditionalOption("myCustomOption", "myCastomValue"); // ignored as non-compliant
options.AddAdditionalOption("browserVersion", "117"); // throws with a message I should use another API

Then the question is what AddAdditionalOption() method is supposed to do?

[diemol]

You can add things like sauce:options or any other options that are W3C compliant.