Re: [pgAdmin][RM4279] : Issue with File Browser Home button - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin][RM4279] : Issue with File Browser Home button
Date
Msg-id CAM9w-_n4PZAy4T2mdk_yRaM8xvKTuakPQWiz=SsZnxARccyjUA@mail.gmail.com
Whole thread Raw
In response to [pgAdmin][RM4279] : Issue with File Browser Home button  (Yogesh Jain <yogesh.jain@enterprisedb.com>)
Responses Re: [pgAdmin][RM4279] : Issue with File Browser Home button  (Yogesh Jain <yogesh.jain@enterprisedb.com>)
List pgadmin-hackers
Hi Yogesh,

Just did an eyeball review. You've included homedir path in the response rendered JS file. You're exposing underlying directory structure to users even in server mode. This is not appropriate. This info should remain at the server and not at the client side.
+        homedir=os.path.expanduser('~'),

On Fri, Apr 24, 2020 at 7:26 PM Yogesh Jain <yogesh.jain@enterprisedb.com> wrote:
Hi Hackers,

Attached is the patch to fix the issue with the file browser Home button.
 
FYI : While clicking on the home button it was pointing to the user's root directory rather than home directory, which is fixed in this patch.

Please review.
PFA.

--
Regards,
Yogesh Jain
8982696654


--
Thanks and Regards,
Aditya Toshniwal
pgAdmin Hacker | Sr. Software Engineer | EnterpriseDB India | Pune
"Don't Complain about Heat, Plant a TREE"

pgadmin-hackers by date:

Previous
From: Domenico Sgarbossa
Date:
Subject: Re: Translators: Release next week
Next
From: Khushboo Vashi
Date:
Subject: [pgAdmin4][Patch] - Improve LDAP Error messages