From 75298fd779b58a189d2c6347f9347243f2c19a6c Mon Sep 17 00:00:00 2001 From: Yaroslav Date: Mon, 30 Mar 2026 17:51:20 +0300 Subject: [PATCH 1/3] added solution --- .github/workflows/test.yml-template | 23 ++++++++++++ package-lock.json | 9 +++-- package.json | 2 +- src/app.js | 2 +- src/createServer.js | 57 ++++++++++++++++++++++++++--- 5 files changed, 82 insertions(+), 11 deletions(-) create mode 100644 .github/workflows/test.yml-template diff --git a/.github/workflows/test.yml-template b/.github/workflows/test.yml-template new file mode 100644 index 0000000..bb13dfc --- /dev/null +++ b/.github/workflows/test.yml-template @@ -0,0 +1,23 @@ +name: Test + +on: + pull_request: + branches: [ master ] + +jobs: + build: + + runs-on: ubuntu-latest + + strategy: + matrix: + node-version: [20.x] + + steps: + - uses: actions/checkout@v2 + - name: Use Node.js ${{ matrix.node-version }} + uses: actions/setup-node@v1 + with: + node-version: ${{ matrix.node-version }} + - run: npm install + - run: npm test diff --git a/package-lock.json b/package-lock.json index e696c03..db0b36a 100644 --- a/package-lock.json +++ b/package-lock.json @@ -12,7 +12,7 @@ "devDependencies": { "@faker-js/faker": "^8.4.1", "@mate-academy/eslint-config": "latest", - "@mate-academy/scripts": "^1.8.6", + "@mate-academy/scripts": "^2.1.3", "axios": "^1.7.2", "eslint": "^8.57.0", "eslint-plugin-jest": "^28.6.0", @@ -1485,10 +1485,11 @@ } }, "node_modules/@mate-academy/scripts": { - "version": "1.8.6", - "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-1.8.6.tgz", - "integrity": "sha512-b4om/whj4G9emyi84ORE3FRZzCRwRIesr8tJHXa8EvJdOaAPDpzcJ8A0sFfMsWH9NUOVmOwkBtOXDu5eZZ00Ig==", + "version": "2.1.3", + "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-2.1.3.tgz", + "integrity": "sha512-a07wHTj/1QUK2Aac5zHad+sGw4rIvcNl5lJmJpAD7OxeSbnCdyI6RXUHwXhjF5MaVo9YHrJ0xVahyERS2IIyBQ==", "dev": true, + "license": "MIT", "dependencies": { "@octokit/rest": "^17.11.2", "@types/get-port": "^4.2.0", diff --git a/package.json b/package.json index 73e02a4..d654180 100644 --- a/package.json +++ b/package.json @@ -18,7 +18,7 @@ "devDependencies": { "@faker-js/faker": "^8.4.1", "@mate-academy/eslint-config": "latest", - "@mate-academy/scripts": "^1.8.6", + "@mate-academy/scripts": "^2.1.3", "axios": "^1.7.2", "eslint": "^8.57.0", "eslint-plugin-jest": "^28.6.0", diff --git a/src/app.js b/src/app.js index 6e754b2..816bc19 100644 --- a/src/app.js +++ b/src/app.js @@ -1,7 +1,7 @@ /* eslint-disable no-console */ 'use strict'; -const { createServer } = require('./createServer'); +const { createServer } = require('./createServer.js'); createServer().listen(5701, () => { console.log(`Server is running on http://localhost:${5701} 🚀`); diff --git a/src/createServer.js b/src/createServer.js index 1cf1dda..fac4246 100644 --- a/src/createServer.js +++ b/src/createServer.js @@ -1,10 +1,57 @@ +/* eslint-disable no-console */ 'use strict'; +const http = require('http'); +const path = require('path'); +const fsp = require('fs/promises'); + function createServer() { - /* Write your code here */ - // Return instance of http.Server class + const server = http.createServer(async (req, res) => { + if (req.url.includes('//')) { + res.statusCode = 404; + res.end('Not Found'); + + return; + } + + const pathname = req.url.slice(1); + + const [action, ...rest] = pathname.split('/'); + const fileName = rest.join('/'); + + if (action !== 'file') { + res.statusCode = 400; + + res.setHeader('Content-Type', 'text/plain'); + res.end('To load file, you need use "/file/fileName"'); + + return; + } + + if (!fileName) { + res.statusCode = 200; + res.setHeader('Content-Type', 'text/plain'); + res.end('Enter the correct path'); + + return; + } + + const realPath = path.join(__dirname, '..', 'public', fileName); + + try { + const file = await fsp.readFile(realPath, 'utf-8'); + + res.statusCode = 200; + res.setHeader('Content-Type', 'text/html'); + res.end(file); + } catch (error) { + res.statusCode = 404; + res.setHeader('Content-Type', 'text/plain'); + res.end('Not Found'); + } + }); + + return server; } -module.exports = { - createServer, -}; +module.exports = { createServer }; From 3caeb9608ebb13ed7b7332daa3205b8cfa9b5801 Mon Sep 17 00:00:00 2001 From: Yaroslav Date: Mon, 30 Mar 2026 18:08:36 +0300 Subject: [PATCH 2/3] fixed --- src/createServer.js | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/createServer.js b/src/createServer.js index fac4246..87f63b5 100644 --- a/src/createServer.js +++ b/src/createServer.js @@ -7,6 +7,8 @@ const fsp = require('fs/promises'); function createServer() { const server = http.createServer(async (req, res) => { + res.setHeader('Content-Type', 'text/plain'); + if (req.url.includes('//')) { res.statusCode = 404; res.end('Not Found'); @@ -17,12 +19,11 @@ function createServer() { const pathname = req.url.slice(1); const [action, ...rest] = pathname.split('/'); - const fileName = rest.join('/'); + const fileName = rest.join('/') || 'index.html'; if (action !== 'file') { res.statusCode = 400; - res.setHeader('Content-Type', 'text/plain'); res.end('To load file, you need use "/file/fileName"'); return; @@ -30,7 +31,7 @@ function createServer() { if (!fileName) { res.statusCode = 200; - res.setHeader('Content-Type', 'text/plain'); + res.end('Enter the correct path'); return; @@ -42,11 +43,10 @@ function createServer() { const file = await fsp.readFile(realPath, 'utf-8'); res.statusCode = 200; - res.setHeader('Content-Type', 'text/html'); res.end(file); } catch (error) { res.statusCode = 404; - res.setHeader('Content-Type', 'text/plain'); + res.end('Not Found'); } }); From 78bfc945bf55f2bc4817f1086f98711a93afee3b Mon Sep 17 00:00:00 2001 From: Yaroslav Date: Mon, 30 Mar 2026 18:17:59 +0300 Subject: [PATCH 3/3] fix --- src/createServer.js | 24 ++++++++++++++++++++---- 1 file changed, 20 insertions(+), 4 deletions(-) diff --git a/src/createServer.js b/src/createServer.js index 87f63b5..711ff24 100644 --- a/src/createServer.js +++ b/src/createServer.js @@ -7,9 +7,18 @@ const fsp = require('fs/promises'); function createServer() { const server = http.createServer(async (req, res) => { - res.setHeader('Content-Type', 'text/plain'); + const MIME_TYPES = { + '.html': 'text/html', + '.css': 'text/css', + '.js': 'application/javascript', + '.json': 'application/json', + '.png': 'image/png', + '.jpg': 'image/jpeg', + '.svg': 'image/svg+xml', + }; if (req.url.includes('//')) { + res.setHeader('Content-Type', 'text/plain'); res.statusCode = 404; res.end('Not Found'); @@ -19,9 +28,10 @@ function createServer() { const pathname = req.url.slice(1); const [action, ...rest] = pathname.split('/'); - const fileName = rest.join('/') || 'index.html'; + const fileName = rest.join('/'); if (action !== 'file') { + res.setHeader('Content-Type', 'text/plain'); res.statusCode = 400; res.end('To load file, you need use "/file/fileName"'); @@ -30,9 +40,9 @@ function createServer() { } if (!fileName) { + res.setHeader('Content-Type', 'text/plain'); res.statusCode = 200; - - res.end('Enter the correct path'); + res.end('To load file, you need use "/file/fileName"'); return; } @@ -42,9 +52,15 @@ function createServer() { try { const file = await fsp.readFile(realPath, 'utf-8'); + const ext = path.extname(fileName); + const contentType = MIME_TYPES[ext] || 'text/plain'; + + res.setHeader('Content-Type', contentType); + res.statusCode = 200; res.end(file); } catch (error) { + res.setHeader('Content-Type', 'text/plain'); res.statusCode = 404; res.end('Not Found');