Обсуждение: [pgAdmin4][Patch]: Do not render security URL in templates if runningin Desktop mode

Поиск
Список
Период
Сортировка

[pgAdmin4][Patch]: Do not render security URL in templates if runningin Desktop mode

От
Murtuza Zabuawala
Дата:
Hi,

PFA patch to fix the issue where we were rendering Security URL's like change password, reset password in HTML templates while running in Desktop mode, we only register these security blueprints in Server mode hence it was failing with BuildError.
RM#2952

Also reverted RM#2922 because it not working as expected and causing runtime to render login screen.


Thanks to Neel for helping me in testing the patch with latest runtime code.


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Вложения

Re: [pgAdmin4][Patch]: Do not render security URL in templates ifrunning in Desktop mode

От
Harshal Dhumal
Дата:
Hi Murtuza,

Moving login related code from under  decorator @app.before_first_request to @app.before_request
will cause runtime user to login on each request. I think we need to find some better way.

-- 
Harshal Dhumal
Sr. Software Engineer

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

On Thu, Dec 14, 2017 at 2:02 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch to fix the issue where we were rendering Security URL's like change password, reset password in HTML templates while running in Desktop mode, we only register these security blueprints in Server mode hence it was failing with BuildError.
RM#2952

Also reverted RM#2922 because it not working as expected and causing runtime to render login screen.


Thanks to Neel for helping me in testing the patch with latest runtime code.


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


Re: [pgAdmin4][Patch]: Do not render security URL in templates ifrunning in Desktop mode

От
Murtuza Zabuawala
Дата:
Yes Harshal, you are right but due to RM#2922 changes now runtime is rendering the Login page.

@Dave,
Please find updated patch, which will also fix the issue of user login on each request in runtime.
RM#2952
RM#2922

--
​ ​
Murtuza

On Thu, Dec 14, 2017 at 2:25 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Murtuza,

Moving login related code from under  decorator @app.before_first_request to @app.before_request
will cause runtime user to login on each request. I think we need to find some better way.

-- 
Harshal Dhumal
Sr. Software Engineer

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

On Thu, Dec 14, 2017 at 2:02 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch to fix the issue where we were rendering Security URL's like change password, reset password in HTML templates while running in Desktop mode, we only register these security blueprints in Server mode hence it was failing with BuildError.
RM#2952

Also reverted RM#2922 because it not working as expected and causing runtime to render login screen.


Thanks to Neel for helping me in testing the patch with latest runtime code.


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



Вложения

Re: [pgAdmin4][Patch]: Do not render security URL in templates ifrunning in Desktop mode

От
Dave Page
Дата:
Thanks, applied.

On Thu, Dec 14, 2017 at 4:00 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Yes Harshal, you are right but due to RM#2922 changes now runtime is rendering the Login page.

@Dave,
Please find updated patch, which will also fix the issue of user login on each request in runtime.
RM#2952
RM#2922

--
​ ​
Murtuza

On Thu, Dec 14, 2017 at 2:25 PM, Harshal Dhumal <harshal.dhumal@enterprisedb.com> wrote:
Hi Murtuza,

Moving login related code from under  decorator @app.before_first_request to @app.before_request
will cause runtime user to login on each request. I think we need to find some better way.

-- 
Harshal Dhumal
Sr. Software Engineer

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

On Thu, Dec 14, 2017 at 2:02 PM, Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:
Hi,

PFA patch to fix the issue where we were rendering Security URL's like change password, reset password in HTML templates while running in Desktop mode, we only register these security blueprints in Server mode hence it was failing with BuildError.
RM#2952

Also reverted RM#2922 because it not working as expected and causing runtime to render login screen.


Thanks to Neel for helping me in testing the patch with latest runtime code.


--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company






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

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