-
Notifications
You must be signed in to change notification settings - Fork 142
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
Support Java 11 #341
Comments
I have tried getting it to work: https://github.com/ArloL/jBrowserDriver/tree/jdk11 I am stuck that CookieManager.setDefault throws an Exception when starting the server. Eclipse also says that method is not accesible. Anyone got an idea? |
Any change on this? I am interested in trying this out but have to use jdk11. |
That’s not the problem. You can run non module code in Java 11. The
problem is a Java 8 specific JavaFX API.
…On Sat, Jan 12, 2019 at 11:37 AM zli-ray ***@***.***> wrote:
@ArloL <https://github.com/ArloL>
Maybe should put a module-info.java into jBrowserDriver to make it a
modular.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#341 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAZcnrKhcs7SaqKtfBV7fHxwOgsRYFw3ks5vCg9RgaJpZM4YxeV0>
.
|
@ArloL @boyarsky Below are my pom.xml, test code and screenshot. |
@ArloL Excellent! Based on the success of @zli-ray , I have also cloned your new branch ArloL/jBrowserDriver/tree/jdk11 and then deployed a new JAR file "jbrowserdriver-1.0.2-SNAPSHOT.jar" utilizing your new XML file "pom.xml". With the new JAR, I can concur with @zli-ray that it is working for JDK 11+ based on the following 2 dependencies:
@ArloL Would you please promote the new version "1.0.2-SNAPSHOT" to production? Thank you very much! Please note that I need to use the flag "--jbd.javaoptions" for the following 11 Java options (which must be separated by tab) for JDK 11+: |
@ArloL As an added bonus, I have successfully tested your new branch ArloL/jBrowserDriver/tree/jdk11 in headless mode using Monocle and can confirm it is working for JDK 11+ based on the following 3 dependencies:
@ArloL Would you please promote the new version "1.0.2-SNAPSHOT" to production? Thank you very much! In headless mode, I need to use the flag "--jbd.javaoptions" for the following 13 Java options (which must be separated by tab) for JDK 11+: |
Any plans to merge this? |
Hi, I updated the code to use openjfx 16 (that is compatible with JDK 11). I created a fork and released it to maven central with a different groupId. Feel free to merge back the changes to this repo. It is available at https://github.com/everit-org/jBrowserDriver |
@balazs-zsoldos could you share the gav coordinates? Gotcha |
@elect86 Works for us. However, we decided to leave jbrowserdriver behind as nobody maintains it and it does not work on Java 17. We use simple selenium webdriver instead (e.g.: chrome webdriver) |
This is related to #285 . Java 11 is a long term support release so more people will be switching to it than Java 9. Also the problem got worse with Java 11. The version of Java FX for Java 11 (open jfx) - https://mvnrepository.com/artifact/org.openjfx gets past the Cookie Manager class not found.
However, Java 11 moves the Robot class from com.sun.glass.ui to javafx.scene.robot. This means that you can’t just use OpenJFX’s version of JavaFX with JDK 11. And since Java FX is tied to the version of Java, you can't use the old Java 8 JavaFX jar.
This means JBrowser Driver is stuck on Java 8 until you explicitly add support for Java 11. I hope this is on the radar. This project is the only headless driver I've found for Java that doesn't require installing anything on the server. Over time, having to stay on Java 8 will become an even bigger problem.
The text was updated successfully, but these errors were encountered: