joerg1985

regarding the PR: It might make sense to handle all Errors the "new" way, not only the two examples?

regarding the "new" way of handling Errors: I am not familiar with the Grid, just my current throughts on this, might be total nonsense :D An Error could let the...

Hi, we did replace the AsyncHttpClient with the Java 11 HttpClient here at work, to get rid of this dependency. The requirements of Selenium are also Java 11, so there...

This branch has a java 11 based HttpClient implementation, in case you decide to update to java 11: https://github.com/joerg1985/selenium/tree/java-11-http-client To make it build the java release version in .bazelrc has...

@DHowett The Json Wire Protocol support has been removed from selenium, see https://github.com/SeleniumHQ/selenium/commit/f28f1df822bea00bbff621f7e843ab46dcb8f85f. It might be time to move on to the W3C wire protocol.

@westsidedev Could you add the following code and check the number of elements in the set, it should be 4: ``` StreamSupport.stream( ServiceLoader.load(CdpInfo.class, CdpVersionFinder.class.getClassLoader()) .spliterator(), false) .collect(Collectors.toSet())); ``` If there...

@westsidedev Okay so the ServiceLoader does not work as expcted. In your pom the target java version is 1.7 and your version in use is 17, this might cause the...

Okay, i think building a jar with all dependencies in it does break the ServiceLoader. I currently have no access to a development environment, so i can not verify this....

I do not think there is mutch that can be done to fix this. The root probleme is that identical files are replaced in the jar. You could exclude the...

@theAnand17 this warning is created via logger here: https://github.com/SeleniumHQ/selenium/blob/aa62f106c736172ff4ccea9399ca005b43140250/java/src/org/openqa/selenium/devtools/CdpVersionFinder.java#L146-L151