Re: libpq parameter parsing problem

Поиск
Список
Период
Сортировка
От Jobin Augustine
Тема Re: libpq parameter parsing problem
Дата
Msg-id CANaTPsr=_2+Ajb-ENdT9QZyk3fjRSrgM+iuZO7vtxe0FU36NGQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: libpq parameter parsing problem  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: libpq parameter parsing problem  (Michael Paquier <michael@paquier.xyz>)
Re: libpq parameter parsing problem  ("David G. Johnston" <david.g.johnston@gmail.com>)
Список pgsql-bugs
Hi Michael,

On Tue, Jan 14, 2020 at 11:33 AM Michael Paquier <michael@paquier.xyz> wrote:
On Mon, Jan 13, 2020 at 08:07:06PM +0530, Jobin Augustine wrote:
> Sorry for the late reply. Attaching a patch.

>     <para>
> -    Percent-encoding may be used to include symbols with special meaning in any
> -    of the <acronym>URI</acronym> parts, e.g. replace <literal>=</literal> with
> -    <literal>%3D</literal>.
> -
> +    Connection <acronym>URI</acronym> need to be encoded with
> +    <ulink url="https://en.wikipedia.org/wiki/Percent-encoding">Percent-encoding</ulink>
> +    if it includes symbols with special meaning in any of its parts.
> +    For example:
> +<programlisting>
> +postgresql://postgres@localhost:5432/postgres?options=-c%20synchronous_commit%3Doff%20-c%20geqo%3Doff
> +</programlisting>
> +    where all <literal>=</literal> are replaced with <literal>%3D</literal> and
> +    space character with <literal>%20</literal>
>     </para>

The reference to wikipedia is nice to have.  A small nit from me is
that I would group the last sentence with the "For example", to give:
"Here is an example where all = are replaced.."
Yes, agree, readability is better with that modification.
Attaching a modified patch.
The first sentence sounds good to me.
--
Michael
 Jobin.
Вложения

В списке pgsql-bugs по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: libpq parameter parsing problem
Следующее
От: Vijay Ingle
Дата:
Сообщение: Oracle To Community Postgresql Migration