Не работает extend параметров http-блока

[vitkarpov]

Создаем новый http-блок:

var block = de.Block.Http(url, {
  method: "POST",
  extend: true
});

далее, хотим доопределить параметры из урла при вызове блока:

block.run(paramsAdditional, context);

paramsAdditional будет добавлен только в том случае, если урл заканчивается на & или ?. Параметр extend при создании экземпляра блока не работает.

Кажется, что это супер неочевидный момент.

[chestozo]

paramsAdditional будет добавлен только в том случае, если урл заканчивается на & или ?

Кажется тут всё ок, вы же руками формируете урл. Можно там указать всё что угодно.

[vitkarpov]

Все так, урл мы формируем сами. Но эти знания, о том как его нужно формировать не находятся нигде, кроме исходного кода.

Кажется, что параметр extend как бы намекает на свое предназначение, но он не работает.

[chestozo]

Ой, я упустил, что есть какой-то параметр extend )

[pasaran]

Погодите, а откуда взяли какой-то параметр extend?
Там есть некий внутренний флаг, но он как раз по урлу строится.

[vitkarpov]

Я подумал про вот этот extend.

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

Поясню в чем дело: иногда хочется сделать пост-запрос, тело запроса передать в json. Поэтому хочется явно пробрасывать параметры для тела, в данном случае подойдет query, а урл не понятно зачем должен заканчиваться на ? или &, просто по логике.

[vitkarpov]

Возможно, имеет смысл всегда прокидывать query в качестве тела запроса для пост-запроса? Не смотря на урл и не вводя дополнительный параметр.

[chestozo]

Возможно, имеет смысл всегда прокидывать query в качестве тела запроса для пост-запроса? > Не смотря на урл и не вводя дополнительный параметр.

Кажется, так не надо делать.
Тело POST запроса это одно, а урл это другое ведь совсем.

[pasaran]

Да, но тот extend берется только вот отсюда: https://github.com/pasaran/descript/blob/master/lib/de.block.js#L561

Просто изначально, дескрипт был очень DSL-ным. Там вообще не предполагалось, что кто-то часто будет использовать явные формы de.*( ... ). Урл это запрос, ? на конце — пробросить запрос и т.д. Не люблю я все эти явные формы, с развесистыми options, громоздко слишком.
Но потом как-то вот туда все сместилось.

Наверное, можно брать его (extend) и из options, если он там есть.

[vitkarpov]

Кажется, так не надо делать.
Тело POST запроса это одно, а урл это другое ведь совсем.

Вообще говоря, да. Хотелось бы расширить АПИ http-блока, например, параметром body. Мне это видется примерно так:

de.http('http://yandex.ru/', {
  method: 'POST',
  query: {
    foo: 'bar'
  },
  body: {
    baz: '123'
  }
})

В итоге будет сделать пост-запрос вида http://yandex.ru/?foo=bar с телом в json: {"baz": "123"}.
В случае не пост-запроса body будет проигнорирован.