-
Notifications
You must be signed in to change notification settings - Fork 11
Features/config worker support and xhr channel support #89
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
base: develop
Are you sure you want to change the base?
Conversation
|
This PR includes dist/* files. Please remove these. Follow up question, should the latest released dist/* file from the last release be on the develop branch? I would lean towards no, but I'm ok with it if that is better... |
|
@dpwspoon Done |
|
Need to fix failing test
|
|
Need to fix karma tests |
|
@jitsni Need to check with you for the possible related issue with encoding on http2.js |
|
Still having issues with gzip test-server using websocket + http2 |
|
@johnnykahalawai Merged with last http2-cache.js changes from develop. @tejaede Note: Worker and ArrayBuffer enabled by default or can be disabled with this: More Example: https://github.com/kaazing/http2-cache.js/blob/features/workerSupport/integration-test/http2-worker-itest.js#L342 cc @dpwspoon |
|
LGTM |
Todo
Usage:
Test result
Reviewer note
Main commit is ece50af
Previous commit to this one are in #88
configuration init worker handler
addConfig handler
sendViaChannel handler
Browser Support
Tested: Chrome, Firefox, Opera, Safari, IE.
Tests results
Test:
With gzip