search

shri-2015-org / 666

666 team's chat =>
http://anonym.club
4 stars 3 forks source link

Wrap socket handlers. Add data validation. #122

Closed pofigizm closed 8 years ago

pofigizm commented 8 years ago

closed #81, closed #102

exah commented 8 years ago

:+1:

deemidroll commented 8 years ago

:+1:

pofigizm commented 8 years ago

@alt-j, @sullenor Как Вам такой вариант? Мнения остальных тоже крайне желательны.

pofigizm commented 8 years ago

Вынес хэндлеры

pofigizm commented 8 years ago

Обновил! Вся работа с сокетами только внутри transport.js, а описание кому / куда / что отправлять осталось в api.handlers.js. После того как @deemidroll закончит #116 можно будет 'убрать' туда хэндлеры. В принципе можно подождать и сразу сделать нормально.

pofigizm commented 8 years ago

Обновил.

Если сегодня Вы пулили ее себе то могут быть конфликты, т.к. я сейчас перезаписал утренний пуш. Если не пулится то -f Вам поможет.

Пока мержить не будем, ждем https://github.com/shri-2015-org/666/pull/116, но комментарии приветствуются.

tempname11 commented 8 years ago

Откомментировал, но ничего критичного нет, и для движения вперед +1