search

dmitryt / jsninja-nodejs-hw

0 stars 0 forks source link

HW #1 http-server #1

Open dmitryt opened 6 years ago

jsninjabot commented 6 years ago

Принято. Мелкое замечание

setStatus(statusCode) {
            if (this.headersSent) {
                return socket.emit('error', new Error('Headers were sent'));
            }

Это неверно. То что отослали заголовки не есть ошибка сокета - сокет же не виноват, поэтому нельзя эмитить ошибку на нем. Более того это синхронная ошибка - поэтому нам достаточно было сделать throw

dmitryt commented 6 years ago

fixed