RocketChat/Rocket.Chat

Avoid deeply nested control flow statements.

Open

                    if (req.headers['if-none-match']) {
                        if (file.etag === req.headers['if-none-match']) {
                            res.writeHead(304); // Not Modified
                            res.end();
                            return;

Avoid deeply nested control flow statements.

Open

                    if (req.headers['if-modified-since']) {
                        const modifiedSince = new Date(req.headers['if-modified-since']);

                        if (
                            (file.modifiedAt instanceof Date && file.modifiedAt > modifiedSince) ||

Avoid deeply nested control flow statements.

Open

                    if (typeof req.headers.range === 'string') {
                        const { range } = req.headers;

                        // Range is not valid
                        if (!range) {

Avoid deeply nested control flow statements.

Open

                    if (typeof req.headers['accept-encoding'] === 'string' && (!file.type || !/^(audio|video)/.test(file.type))) {
                        const accept = req.headers['accept-encoding'];

                        // Compress with gzip
                        if (accept.match(/\bgzip\b/)) {

Avoid too many return statements within this function.

Open

            return;

Avoid too many return statements within this function.

Open

                                return;

Avoid too many return statements within this function.

Open

            return;

Avoid too many return statements within this function.

Open

            return;

Avoid too many return statements within this function.

Open

                            return;

Avoid too many return statements within this function.

Open

                            return;

Avoid too many return statements within this function.

Open

            return;

Similar blocks of code found in 2 locations. Consider refactoring.

Open

    fs.stat(path, (err) => {
        if (err) {
            // Create the temp directory
            mkdirp(path, { mode })
                .then(() => {

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                        if (accept.match(/\bgzip\b/)) {
                            headers['Content-Encoding'] = 'gzip';
                            delete headers['Content-Length'];
                            res.writeHead(status, headers);
                            ws.pipe(zlib.createGzip()).pipe(res);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                        if (accept.match(/\bdeflate\b/)) {
                            headers['Content-Encoding'] = 'deflate';
                            delete headers['Content-Length'];
                            res.writeHead(status, headers);
                            ws.pipe(zlib.createDeflate()).pipe(res);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                ws.on('error', (err) => {
                    store.onReadError.call(store, err, fileId, file);
                    res.end();
                });

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                rs.on('error', (err) => {
                    store.onReadError.call(store, err, fileId, file);
                    res.end();
                });

• Read up
  Read up