Skip to content
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

addRequestHandler override the default CefRequestHandler #1

Closed
KevinnZou opened this issue Nov 30, 2023 · 3 comments
Closed

addRequestHandler override the default CefRequestHandler #1

KevinnZou opened this issue Nov 30, 2023 · 3 comments

Comments

@KevinnZou
Copy link

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

@DatL4g
Copy link
Owner

DatL4g commented Jan 7, 2024

8ebc5fe

@DatL4g DatL4g closed this as completed Jan 7, 2024
@KevinnZou
Copy link
Author

@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
Copy link
Author

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants