Почему не работает переадресация после POST запроса в Node.js?

Question

[i_surin]

честно искал, ничего не нашёл, голову ломаю понять не могу в чём дело
форма в html файле ссылается на обработчик

app.post('/complete', function(req, res) {
	completeFile = "]}";

	fs.appendFile(projectName + '.json', completeFile, function(err) {
		if (err) console.error(err);
		console.log('file completed');
		res.writeHead(301, {
			Location: "result.html"
		});
		res.end()
	});
})

по идее res.writeHead должен перекинуть на страницу result.html, которая лежит в той же папке, что и файл server.js (с node-кодом) и с файлом index.html, на котором форма

Как сделать чтобы оно работало?

Answer 1

[#алгоптимизируй #отботизируй]

асинхронный вызов что ли? Так не пойдет(наверное)

Comments

[i_surin]

а как нужно сделать? мне просто нужно перейти на эту страницу после нажатия кнопки в форме (имя возможно будет меняться пользователем)

[i_surin]

просто он как бы вообще никак на эту страницу не переходит, я с таким в вордпресс сталкивался, но там нужно было просто встроенную функцию перед ссылкой вставить, как здесь сделать не представляю

[#алгоптимизируй #отботизируй]

тут посмотрите
stackoverflow.com/questions/25467151/node-js-wait-...
Я не уверен,что прав, но складывается впечатление,что происходит запись в файл и дальше асинхронно происходит вызов коллбэка,само собой на тот момент заголовки уже отданы клиенту, то есть нужно просто передавать заголовки до или ждать записи. повторюсь не уверен.

[#алгоптимизируй #отботизируй]

ждать=блокирующий wait

[i_surin]

С Sharp: ну в целом то вы правы, всё так и есть, но по всей видимости я не понимаю как оно работает, ведь заголовки я передаю уже после записи в файл, разве нет? ну и опять же если я заголовки передаю после res.end, то он это просто игнорирует и выводит мне пустую страницу. Вопрос именно в том, куда нужно сунуть какую то конкретную строчку, я этого понять не могу, по ссылке выше посмотрел, не нашёл для себя решения

[#алгоптимизируй #отботизируй]

i_surin: а в броузере в консоли разработчиков смотрели,что выдается?
Я просто не знаю, как работает res.end и когда его можно вызывать. Погуглю.

[i_surin]

С Sharp: в консоли 404, по адресу localhost:3000/result.html

[#алгоптимизируй #отботизируй]

А кто мешает вызывать writehead and end before appendfile ?

[i_surin]

С Sharp: вызываю, та же ошибка

[i_surin]

app.post('/complete', function(req, res) {
completeFile = "]}";

res.writeHead(301, {
Location: "result.html"
});
res.end()

fs.appendFile(projectName + '.json', completeFile, function(err) {
if (err) console.error(err);
console.log('file completed');
});
})

вот так же?

[#алгоптимизируй #отботизируй]

все так же- это вопрос или факт?

[i_surin]

С Sharp: это вопрос по тому коду который выше кинул, там я вызываю writehead до appendfile, но всё так же ошибка 404

[#алгоптимизируй #отботизируй]

i_surin: просмотрел сообщение про 404 )))))))))))))))))) Это надо было в вопросе писать.
Тогда проблема с путями.Какой адрес страницы с формой?

[#алгоптимизируй #отботизируй]

Вижу index.html.

[i_surin]

ну если на компьютере, то E:\desktop\project\site\index.html, а так он сразу на неё кидает после перехода на localhost:3000

[#алгоптимизируй #отботизируй]

А какой ПОЛНЫЙ путь к странице формой в броузере?

[#алгоптимизируй #отботизируй]

localhost:3000/index.html

это открывается?

[i_surin]

С Sharp: просто ещё не до конца разобрался, когда я запускаю сервер я ввожу команду node server.js, то есть обращаюсь к файлу server.js, в котором у меня уже подключается файл index.html со всеми формами, то есть возможно localhost это файл server.js, но в любом случае он лежит в той же директории где и остальные

[i_surin]

С Sharp: ну и опять же если вместо result.html я ставлю например ссылку на эту страницу или куда либо ещё, то он переадресовывает

[i_surin]

С Sharp: localhost:3000/index.html не открывается

[#алгоптимизируй #отботизируй]

i_surin: а у вас настроена отдача статических файлов, кроме index.html ?

[#алгоптимизируй #отботизируй]

Хранить node скрипт в папке с html - странная история, может так и не можно.....Так можно случайно его содержимое и клиенту выдать,как-то не безопасно. На nodejs не пишу, опять же,не знаю.

[i_surin]

С Sharp: нет

сам index.html сделан так:

var html = fs.readFileSync('./index.html');

app.get('/', function(req, res) {
res.writeHeader(200, {
"Content-Type": "text/html"
});
res.end(html);
});

то есть мне нужно сделать ещё таких переменных по типу html только с другими нужными мне файлами и по местам их рассовать?

[#алгоптимизируй #отботизируй]

i_surin:
app.use(express.static('public'));
app.use(express.static('uploads'));
app.use(express.static('files'));
expressjs.com/ru/guide/using-middleware.html

Нужно настроить вывод статики я так понимаю.

[i_surin]

С Sharp: да я обычно тоже не пишу, по надобности пришлось, вот и не могу разобраться в мелочах подобных)

[#алгоптимизируй #отботизируй]

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

[i_surin]

С Sharp: в моём случае получается, что это html, к которому подключены свои скрипты, стили и картинки, а некоторые данные на него добавляются из файла, который пользователь заполняет с помощью формы на странице index.html, а нажимая кнопку завершить его просто кидает на страницу result.html, на которой уже всё готово + есть забитые им данные

[#алгоптимизируй #отботизируй]

i_surin: Если result отдается как статический файл,значит он должна быть настроена отдача статики.
Выше я кинул пример настройки статики для папок, вроде бы.
Я правда не вижу смысла вообще в этом, можно просто сгенерировать в ответе содержимое сразу,без переадресаций на перезаписываемый файл, который перепишется после каждой отправки.
Имя может делать уникальное хотя бы у файла или типа того.

[#алгоптимизируй #отботизируй]

значит для него должна***

[i_surin]

С Sharp: ну имя я как раз и планирую делать уникальное, там есть переменная projectName из которой я и хотел брать имя с дописыванием просто расширения .html, только как тогда генерировать его сразу в ответе? это было бы намного лучше и правильней)

[#алгоптимизируй #отботизируй]

i_surin: В end можно передавать строчку,которая будет разметкой html страницы (и кодировку)
request.end([data][, encoding][, callback])

[#алгоптимизируй #отботизируй]

i_surin: есть костыльный метод, прочитать файл-шаблон и через реплэйс заменить только то,что меняется)
template = readfile();
html = template.replace ("%%bonus%%", current_bonus)
end(html)

[#алгоптимизируй #отботизируй]

with html_encode in nodejs )

[i_surin]

С Sharp: теперь я запутался ещё больше и запрос вообще отвечает тем, что не может ничего отобразить :D

[#алгоптимизируй #отботизируй]

i_surin: покажите код текущий

[i_surin]

С Sharp: в итоге можно же сделать с помощью этого: request.end([data][, encoding][, callback])? то есть в дату я сую переменную, которая получает мою html страницу, в encoding пишу utf8 и ещё что то в колбэк, или так его оставить?

[#алгоптимизируй #отботизируй]

i_surin: в data нужно строчку(html) передать,в callback можно ничего не передавать. То, что в квадратных скобках -не обязательные параметры. Они тут все такие.

[#алгоптимизируй #отботизируй]

аналог
request.write(chunk[, encoding][, callback])#

Sends a chunk of the body. By calling this method many times, the user can stream a request body to a server--in that case it is suggested to use the ['Transfer-Encoding', 'chunked'] header line when creating the request.

The chunk argument should be a Buffer or a string.

[i_surin]

С Sharp: в командной строке он ругается на то, что req.write is not a function

[i_surin]

С Sharp:

делаю так:

result = fs.readFileSync('result.html');

app.post('/complete', function(req, res) {
completeFile = "]}";

fs.appendFile(projectName + '.json', completeFile, function(err) {
if (err) console.error(err);
console.log('file completed');
req.write(result,'utf8');
});
})

[#алгоптимизируй #отботизируй]

res или req ?

[i_surin]

С Sharp: вот чёрт, заменил на res и всё заработало, спасибо добрый человек)

[#алгоптимизируй #отботизируй]

i_surin: пожалуйста)