-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Upgrade version of Selenium #17045
Upgrade version of Selenium #17045
Conversation
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DOTNET-SELENIUMWEBDRIVER-6062317
…b31049e79 [Snyk] Security upgrade Selenium.WebDriver from 3.5.0 to 4.14.1
Changing Version of Selenium WebDriver
The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-DOTNET-SELENIUMWEBDRIVER-6062317
…b31049e79 [Snyk] Security upgrade Selenium.WebDriver from 4.10.0 to 4.14.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey there! Selenium 4 is incompatible with WinAppDriver (the testing solution we use) and it will therefore not work with our project. If you're going to upgrade dependencies--especially between major versions--please make sure that you've completed a local build and test cycle!
Then which version does it support? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think now WinAppDriver supports Selenium.
Thanks for this! We're moving nuget feeds, so we will re-evaluate our dependencies at that time. |
I wonder if Selenium 3.142.7 works with WinAppDriver. |
I'm not involved with WinAppDriver any longer, and I cannot vouch for the veracity of any information generated by that... thing. |
Do you know anyone who is involved with WinAppDriver. |
I do not, I am sorry. |
Summary of the Pull Request
This reduces vulnerabilities.
References and Relevant Issues
This closes #17048
Detailed Description of the Pull Request / Additional Comments
Validation Steps Performed
PR Checklist