Akryum / Akryum/meteor-socket-io

Updates required because of version changes

Ouverte
#3 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
25
Forks
7
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

This is a great example, thank you so much - I couldn't have got my code working without it.
I had to change a few things in the code because of version changes.

1. meteor-node-stubs now uses stream-http instead of http-browserify
[https://github.com/meteor/node-stubs/issues/14](https://github.com/meteor/node-stubs/issues/14)

So the 'hack' fix to Response in the client code isn't needed, and in fact causes an error:

`Error: Cannot find module 'meteor-node-stubs/node_modules/http-browserify/lib/response'`

I had to update meteor-node-stubs because the version of stream-http it included used the Buffer polyfill which is no longer included in Meteor by default:

[https://github.com/meteor/meteor/pull/8327/commits/6f75eae1353b187e358ced331e5f8e3b91cda08d](https://github.com/meteor/meteor/pull/8327/commits/6f75eae1353b187e358ced331e5f8e3b91cda08d)

2. On the server, I had to change:

`import http from 'http';`

to:

`const http = require('http');`

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.