search

DatL4g / KCEF

Kotlin implementation of jcefmaven with more modern setup and depending on JetBrains/jcef
https://datl4g.github.io/KCEF/
Apache License 2.0
28 stars 5 forks source link

addRequestHandler override the default CefRequestHandler #1

Closed KevinnZou closed 6 months ago

KevinnZou commented 7 months ago

As we discussed in this issue. addRequestHandler will override the default CefRequestHandler and block the future request.

DatL4g commented 6 months ago

8ebc5fee0281b52051474cb9ccbb645b71605852

KevinnZou commented 5 months ago

@DatL4g It appears that you just provided a default CefResourceRequestHandler, but the default CefRequestHandler is still missing. So should the same problem still exist if we just call addRequestHandler which will override the default implementation?

I have tested it with the latest KECF version and confirmed that the problem still exists. You can find the test code here.

KevinnZou commented 5 months ago

@DatL4g It appears that you just provided a default CefResourceRequestHandler, but the default CefRequestHandler is still missing. So should the same problem still exist if we just call addRequestHandler which will override the default implementation?

I have tested it with the latest KECF version and confirmed that the problem still exists. You can find the test code here.

@DatL4g Are there any updates on this issue? We need to override onBeforeBrowse to intercept the Url request.