убили киллерфичу rethinkdb

change streams выглядит вкусно

еееее, классно, а то в метеоре через oplog и poll observe работает, а тут уже из коробки такая фича

убили киллерфичу rethinkdb

так он вроде не особо живой уже был, у них там тяжкие времена настали с поддержкой, когда я последний раз смотрел на него. хотя может чего изменилось уже

так он вроде не особо живой уже был, у них там тяжкие времена настали с поддержкой, когда я последний раз смотрел на него. хотя может чего изменилось уже

Насколько я знаю они давно уже "не живые" и продукт поддерживается сообществом. Или около того

Господа, написал тут маленького телеграм бота на ноде, может кто мне код ревью провести?

Не вижу никакого кода

Где ссылка на гитхаб?

https://github.com/WillDonut/telegram-bot-which-hates-stickers Я хотел в личку кинуть, но ладно опозорюсь на 2к человек.

ну такое, зачем миксить колбеки и промисы? почему асинк\авейт не заюзать? по самому стилю кода тоже бы все к единому целому привести

https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/tools.js#L4 тут for поменять на одну строчку по хорошему

https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/tools.js#L4 тут for поменять на одну строчку по хорошему

Ну на мап, я знаю, но именно там я это почти осознано сделал.

там где if (err) должен быть throw по хорошему

events.js L100 то что внутри if вынеси лучше в переменную с понятным названием, а то непонятно при каких случаях условие должно сработать

там где if (err) должен быть throw по хорошему

Понял.

https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/commands.js#L74 тут не понятно почему пустой массив аж на три строки? "packs": []

ну такое, зачем миксить колбеки и промисы? почему асинк\авейт не заюзать? по самому стилю кода тоже бы все к единому целому привести

Ну скажем так, я осознано их не юзал, чтоб попробовать пованильнее, как бы смешно это не звучало.

так тогда все пованильнее фигач

по consistency - где-то в коде используется строгое сравнение, где-то нестрогое

https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/commands.js#L41 тут !=, семь строк ниже !==

https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/commands.js#L41 тут !=, семь строк ниже !==

Сейчас исправлю, спасибо.

https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/tools.js#L31 подобные конструкции можно менять на module.exports = { isBotAdmin, isUserAdmin }

и много странных пустых строк, стоит юзать линтер или что-то подобное https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/commands.js#L35

там где if (err) должен быть throw по хорошему

А вдруг он хочет писать в функциональном стиле и позитивно ресолвить все ?

тогда бы вместо for был бы map :)))

Всем привет. Работаю в reg.ru разработчиком. Знаю много всего про Perl, архитектуру веб-проектов, удаленную командную разработку, новичок в ноде. Надеюсь, сообщество подскажет и наставит на путь истиный в изучении ноды. Сам и Москвы. Узнал из поиска. #whois

Подписка на #whois

Всем привет. Работаю в reg.ru разработчиком. Знаю много всего про Perl, архитектуру веб-проектов, удаленную командную разработку, новичок в ноде. Надеюсь, сообщество подскажет и наставит на путь истиный в изучении ноды. Сам и Москвы. Узнал из поиска. #whois

Вы плохие люди Продавать домены задешево, а потом в 6 раз дороже продление без уведомлений- это подстава

Это не ко мне, а к бизнесу ))

это все есть в соглашении, которое вы принимаете ставя галочку (:

и о всех ценах заранее сообщают

это все есть в соглашении, которое вы принимаете ставя галочку (:

Тем не менее, хороший бизнес дает пояснение заранее

тогда бы вместо for был бы map :)))

Да

Как я понимаю здесь чат про ноду, а не про домены

Я уже сваливаю с reg.ru обратно

☝️

Всем привет, работаю в МТС ИТ руководителем проектов по разработке, много лет занимался разработкой (в основном PHP). С NodeJS с 2013 года. Всем добра ) #whois

Подписка на #whois

Вы плохие люди Продавать домены задешево, а потом в 6 раз дороже продление без уведомлений- это подстава

Кстати, да, меня это выбесило один раз

Сейчас исправлю, спасибо.

И да eslint-config-airbnb, если не используешь

И да eslint-config-airbnb, если не используешь

Я читал про него, но решил пованильнее. в следующий раз буду что-то писать и дебагать не через console.log, хех.

Я читал про него, но решил пованильнее. в следующий раз буду что-то писать и дебагать не через console.log, хех.

Пованильнее? Чем руководствовался, когда решал пованильнее?

он никак не мешает ванильности :D

Подскажите как справиться с падением express.js. На ошибке TypeError: Cannot read property 'match' of undefined падает. Обработчик ошибок по инструкции из доки официальной. Но чего-то он не обрабатывает нифига, хотя по идее любой нормальный веб-сервер такого рода ошибки должен нормально обрабатывать и не падать

Пованильнее? Чем руководствовался, когда решал пованильнее?

Тогда тут скорее подходит слово попроще.

Подскажите как справиться с падением express.js. На ошибке TypeError: Cannot read property 'match' of undefined падает. Обработчик ошибок по инструкции из доки официальной. Но чего-то он не обрабатывает нифига, хотя по идее любой нормальный веб-сервер такого рода ошибки должен нормально обрабатывать и не падать

Как выводишь ошибки?

где-то вызывается match на undefined. Скорее всего на вашей части кода, а не на express

Тогда тут скорее подходит слово попроще.

Лучше как надо, а не проще страдай Я не шучу

И да eslint-config-airbnb, если не используешь

NOOOOOOO

app.use(function errorHandler(err, req, res, next) { if ( err ) { console.error(err.stack); res.status(500).send('Something broke!'); return; } next(); });

NOOOOOOO

Идите вы со своим отсутствием ; !)

Идите вы со своим отсутствием ; !)

Идите вы со своими ; !)

Функция в мидварине сама

это какая версия express? в последней только три аргумента в middleware

А, вижу

Лучше как надо, а не проще страдай Я не шучу

Ну когда я начинаю что-то новое учить, я стараюсь использовать инструменты как можно проще. Чтоб в ходе самого процесса изучения хотя бы с ними у меня не возникло трудностей.

app.use(function errorHandler(err, req, res, next) { if ( err ) { console.error(err.stack); res.status(500).send('Something broke!'); return; } next(); });

Пример того, что в консоль падает

"express": "4.14.0",

и много странных пустых строк, стоит юзать линтер или что-то подобное https://github.com/WillDonut/telegram-bot-which-hates-stickers/blob/master/src/commands.js#L35

Спасибо.

"express": "4.14.0",

на какую строку указывает при падении?

TypeError: Cannot read property 'match' of undefined at Function.gqlquery (/opt/***/middleware/logger.js:19:16) at /opt/***/middleware/logger.js:11:20 at Array.logRequest (/opt/***/node_modules/morgan/index.js:122:18) at listener (/opt/***/node_modules/on-finished/index.js:169:15) at onFinish (/opt/***/node_modules/on-finished/index.js:100:5) at callback (/opt/***/node_modules/ee-first/index.js:55:10) at ServerResponse.onevent (/opt/***/node_modules/ee-first/index.js:93:5) at emitNone (events.js:110:20) at ServerResponse.emit (events.js:207:7) at onFinish (_http_outgoing.js:733:10) at _combinedTickCallback (internal/process/next_tick.js:131:7) at process._tickDomainCallback (internal/process/next_tick.js:218:9) [nodemon] app crashed - waiting for file changes before starting...

logger.js import morgan from 'morgan'; module.exports = function() { return morgan(function (tokens, req, res) { return [ '==>', tokens.method(req, res), tokens.url(req, res), tokens.status(req, res), ' : ', tokens.gqlquery(req, res), '-', tokens['response-time'](req, res), 'ms', ].join(' '); }); }; morgan.token('gqlquery', function (req, res) { let list = req.body.query.match(/(query[^{]+) \{/); if ( list ) { return list[0]; } return ''; });

logger.js import morgan from 'morgan'; module.exports = function() { return morgan(function (tokens, req, res) { return [ '==>', tokens.method(req, res), tokens.url(req, res), tokens.status(req, res), ' : ', tokens.gqlquery(req, res), '-', tokens['response-time'](req, res), 'ms', ].join(' '); }); }; morgan.token('gqlquery', function (req, res) { let list = req.body.query.match(/(query[^{]+) \{/); if ( list ) { return list[0]; } return ''; });

req.body.query не задан

+

То что оно там падает это норм. Непонятно почему express крашится

он должен вернуть 500

ERROR: S client not available

или я что-то не понимаю

он должен вернуть 500

Ошибка происходит не внутри экспресса

оберните в try catch query.match и падать не будет, тк падает весь процесс

или я что-то не понимаю

У тебя контекст функции не в области экспресса лежит ошибка падает через логгер

Те получается любой код middleware и обработчиков роутов нужно в try заворачивать?

Те получается любой код middleware и обработчиков роутов нужно в try заворачивать?

Нет Как ты свой обработчик в приложении используешь?

app.use

Сек, внимательнее код прочту

везде где вызов метода над undefined (который не имеет методов) - будет креш

Сам хендлер ошибок module.exports = function errorHandler(err, req, res, next) { if ( err ) { console.error(err.stack); res.status(500).send('Something broke!'); return; } next(); }

поэтому либо надо проверять на undefined, либо оборачивать в try catch

и в основном скрипте app.use(errorHandler) в конце

например If (foo && foo.bar && foo.bar.xyz) , либо try catch на мидлвар

А в express middleware не обарачивается в try разве?

Те получается любой код middleware и обработчиков роутов нужно в try заворачивать?

так что да, любой. Просто обычно там уже все предусмотрено и там undefined нигде не вылезет

и в основном скрипте app.use(errorHandler) в конце

У тебя morgan.token объявлен вне экспресса и все ошибки пудут падать вне экспресса

что значит вне экспресса?

У него область видимости не наследуется от функции, которая потом будет передана в экспресс

У тебя код лежит на уровне модуля Модуль- тоже функция

вот блин

Колбек внутри morgan.token будет наследовать контекст модуля

А в express middleware не обарачивается в try разве?

нет

Те получается если в любом стороннем модуле будет подобная бага, то экспресс рухнет и ничего с этим не сделаешь?

падает не экспресс

Те получается если в любом стороннем модуле будет подобная бага, то экспресс рухнет и ничего с этим не сделаешь?

Не всегда Можно написать так, чтобы область видимость лежала в области экспресса

Для этого тебе нужно в экспресс передать функцию, которая выполнит все это

wrapper

Есть какой-то глобальный хендлер в ноде, с помощью которого можено ошибки обрабатывать и не убирвать процесс?

Есть какой-то глобальный хендлер в ноде, с помощью которого можено ошибки обрабатывать и не убирвать процесс?

Есть, но это херовое решение

try catch ?:)

Как факт- process.on('uncaughtException', (err) => {})