AstralTransRocketries/node-minecraft-protocol
1
0
Fork 0
mirror of https://github.com/unmojang/node-minecraft-protocol.git synced 2025-09-29 14:13:45 -04:00
Code Issues Packages Projects Releases Wiki Activity

Update autoVersion.js to emit errors instead of throwing

Browse Source
This commit is contained in:
Ethan Davis 2018-01-15 20:30:50 -08:00
parent eef401c1c8
commit 01f8342f9e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/client/autoVersion.js
View File

@ -11,7 +11,7 @@ module.exports = function(client, options) {
debug('pinging',options.host);
// TODO: use 0xfe ping instead for better compatibility/performance? https://github.com/deathcap/node-minecraft-ping
ping(options, function(err, response) {
if (err) throw err; // hmm
client.emit('error',err);
debug('ping response',response);
// TODO: could also use ping pre-connect to save description, type, max players, etc.
const motd = response.description;
@ -30,7 +30,7 @@ module.exports = function(client, options) {
.sort(function (a, b) { return b.version - a.version })
.concat(minecraft_data.postNettyVersionsByProtocolVersion["pc"][protocolVersion]||[])
if (versions.length === 0) {
throw new Error(`unsupported/unknown protocol version: ${protocolVersion}, update minecraft-data`);
client.emit('error',`unsupported/unknown protocol version: ${protocolVersion}, update minecraft-data`);
}
const minecraftVersion = versions[0].minecraftVersion;