-
Notifications
You must be signed in to change notification settings - Fork 52
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
Make use of grpc-web interceptor #68
Comments
BTW, improbable also supports interceptors, so all users might benefit from this great dev tool. |
Yea; this is something that I would like to get to doing at some point. Happy for PRs if someone has the time to look into this? |
pull request for interceptors #79 |
merged this change into my fork's master and updated the readme to reflect the change. I'm dropping a link here for anyone that finds this and wants to use native gRPC-Web interceptors: |
@jrapoport So does your version work with https://github.com/improbable-eng/grpc-web? If so, do you need to do anything to make sure that the browser has a specific version of devtools (which supports one client library over the other) installed? |
no clue. it was written for https://github.com/grpc/grpc-web interceptors. the version i wrote isn't published so you have to install manually and disable other versions. it might work with improbable interceptors if they are functionally the same, but i've never tried it. |
The following code works for me for improbable-eng
|
Created this PR with several lines of code to fix this issue. |
Now, when
grpc-web
officially supports interceptors , would be great to see this extension uses them.The text was updated successfully, but these errors were encountered: