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

RFE: remove pusher.com dependency - Merged #1

Conversation

adamlwgriffiths
Copy link

@adamlwgriffiths adamlwgriffiths commented Nov 3, 2020

Merged version of atom#50.

Regenerated packages-lock.json.
There were a number of broken, deprecated, or vulnerable packages previously.

Server runs, but have not tested with client as I don't have GH OAuth setup nor the custom CRDT / Client installed.

jasonrudolph and others added 16 commits July 24, 2018 14:31
Upgrade to latest patch version of Express 4.15.x
Upgrade dependencies to address `npm audit` findings
When running tests from the teletype package we instantiate a test
server to assert the end to end integration between client and server.

This is problematic on Electron v3 because of a crash that happens due
to the bugsnag module. Since this feature is not necessary in tests we
decided to allow disabling it to work around the problem.

Refs: atom/teletype#438
Co-Authored-By: Antonio Scandurra <[email protected]>
Add flag to disable bugsnag on the test server.
Removes `client_id` and `client_secret` query params from auth headers
@nonameentername nonameentername merged commit b4f09ac into nonameentername:private-solution Jan 4, 2021
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

Successfully merging this pull request may close these issues.

5 participants