From: | Murtuza Zabuawala <murtuza(dot)zabuawala(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [RM#3998] Custom encode forward slashes in the grid title |
Date: | 2019-03-01 12:41:12 |
Message-ID: | CAKKotZQ3Z_hq8zD52NsrS7yxexpKa=zKsCBzt0RJKb6U+iq0Xw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hello Aditya,
Yes, that is a better approach, PFA updated patch.
Thanks,
Murtuza
On Fri, Mar 1, 2019 at 4:35 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
> That's a good idea. Seems much more robust.
>
> On Fri, Mar 1, 2019 at 11:02 AM Aditya Toshniwal <
> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>
>> Hi Murtuza,
>>
>> I would suggest adding a new url param like "?fslash=12,15", which says
>> where to put the forward slashes.
>>
>> So if server name is "PostgreSQL/11" - connecting to postgres database
>> with postgres username:
>> Old URL:
>> http://localhost:5005/datagrid/panel/3400871/true/postgres%2Fpostgres%40PostgreSQL%2F11?query_url=&server_type=pg
>>
>> New URL:
>> http://localhost:5005/datagrid/panel/3400871/true/postgrespostgres%40PostgreSQL11?query_url=&server_
>> <http://localhost:5005/datagrid/panel/3400871/true/postgrespostgres%40PostgreSQL11?query_url=&server_type=pg&>
>> type=pg&fslash=8,27
>>
>> Value extracted: postgrespostgres(at)PostgreSQL11
>> Using fslash param, put the forward slashes in the mentioned positions:
>> postgres/postgres(at)PostgreSQL/11
>>
>
>
>>
>>
>> On Fri, Mar 1, 2019 at 4:07 PM Murtuza Zabuawala <
>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>
>>>
>>> On Fri, Mar 1, 2019 at 3:44 PM Aditya Toshniwal <
>>> aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:
>>>
>>>> Hi Murtuza,
>>>>
>>>> Will this work if server name contains - !PG! ?
>>>>
>>>
>>> No, it won't and it's a custom signature we can finalize it whatever
>>> team thinks will work if the functionality is working properly.
>>>
>>>
>>>>
>>>> On Fri, Mar 1, 2019 at 12:18 PM Murtuza Zabuawala <
>>>> murtuza(dot)zabuawala(at)enterprisedb(dot)com> wrote:
>>>>
>>>>> Hello,
>>>>>
>>>>> PFA patch for the issue RM#3998. If the URL contains the forward slash
>>>>> value in the query tool title and if pgAdmin4 is behind Apache server then
>>>>> it fails to open to query tool and the user gets an error 404 because by
>>>>> default Apache does not allow encoded slashes in the URL and return 404.
>>>>>
>>>>>
>>>>> --
>>>>> Regards,
>>>>> Murtuza Zabuawala
>>>>> EnterpriseDB: 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"
>>>>
>>>
>>
>> --
>> 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
>
Attachment | Content-Type | Size |
---|---|---|
RM_3998_v1.diff | application/octet-stream | 6.7 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2019-03-01 12:49:09 | Re: Result grid rendering (was: Re: [pgAdmin4][Pattch] - RM #3673 - "Download as .csv" F8 does NOT work when one of joined files is a TEMPORARY file) |
Previous Message | Dave Page | 2019-03-01 12:39:06 | Re: [pgAdmin4][RM4041] Graphical explain plan is not rendering for a query |