Re: [pgAdmin4][RM3371] Ping endpoint still send a pg4a_session cookie

Поиск
Список
Период
Сортировка
От Aditya Toshniwal
Тема Re: [pgAdmin4][RM3371] Ping endpoint still send a pg4a_session cookie
Дата
Msg-id CAM9w-_kTDprObxK45p+YRAD=D_VKtyNotBAMi2FShey6qDp-6Q@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [pgAdmin4][RM3371] Ping endpoint still send a pg4a_session cookie  (Dave Page <dpage@pgadmin.org>)
Ответы Re: [pgAdmin4][RM3371] Ping endpoint still send a pg4a_session cookie  (Dave Page <dpage@pgadmin.org>)
Список pgadmin-hackers
Дерево обсуждения
Hi Dave,

I tested with apache2 and it works fine. Below is the log:

adityatoshniwal@ubuntu:~/.pgadmin/sessions$ ls -ltr
total 0
adityatoshniwal@ubuntu:~/.pgadmin/sessions$ curl -vvv "http://pgadmin.local/pgadmin/misc/ping"
*   Trying 127.0.0.1...
* Connected to pgadmin.local (127.0.0.1) port 80 (#0)
> GET /pgadmin/misc/ping HTTP/1.1
> Host: pgadmin.local
> User-Agent: curl/7.47.0
> Accept: */*
>
< HTTP/1.1 200 OK
< Date: Wed, 04 Jul 2018 05:36:37 GMT
< Server: Apache/2.4.18 (Ubuntu)
< Set-Cookie: pga4_session=d9ec42c2-7a9d-4e7a-abae-e510a7bee94e!7GpTdrxChvZaPCud1IZS4BdqwgU=; Expires=Thu, 05-Jul-2018 11:06:37 GMT; HttpOnly; Path=/
< Content-Length: 4
< Content-Type: text/html; charset=utf-8
<
* Connection #0 to host pgadmin.local left intact
PINGadityatoshniwal@ubuntu:~/.pgadmin/sessions$ ls -ltr
total 0
adityatoshniwal@ubuntu:~/.pgadmin/sessions$


On Mon, Jul 2, 2018 at 8:29 PM, Dave Page <dpage@pgadmin.org> wrote:
Hi

I'm not in a position to test at the moment, but can you confirm this will work if the root of pgAdmin is at (for example) /pgadmin? So the actual root is /pgadmin/misc/ping ?

On Mon, Jul 2, 2018 at 1:35 PM, Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch for fixing RM3371 where /misc/ping service generate session file for each call and so cannot be used frequently.
The patch is to skip session file generation and session caching for the URLs provided in SESSION_SKIP_PATH list config parameter. pg4a_session_cookie value will still be generated but nothing will be stored at the backend.
Also, I have separated the garbage collection code in current ping service to a new url /misc/cleanup. /misc/ping will be purely for is alive check.

Request you to kindly review.

--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"



--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



--
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"

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

Предыдущее
От: Dave Page
Дата:
Сообщение: pgAdmin 4 commit: Build PG as a dep on macOS
Следующее
От: Aditya Toshniwal
Дата:
Сообщение: Re: [pgAdmin4][RM3294] User need to reset the layout to see thechanged preferences parameters