-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[Notifier] Add Pusher.com bridge #53267
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
Conversation
chore: remove @internal annotation from tests chore: add footer to README.md chore: apply PR comments
feat: configure pusher notifier on framework bundle
fix: drop sensitive information from toString method on transport
f034b8b
to
974c996
Compare
I have mixed feelings about this bridge, code looks OK now after some fixes, but injecting the build client feels weird. In other bridges we do not depend on third party libraries. If someone want to take over and need this bridge, feels free to use this PR as a starting point. Closing here |
I haven't read the code but we don't want bridges that "just" wrap other clients. |
Replaces #48203 because
Allow edits from Maintainer
box is unchecked