Re: [pgAdmin][PATCH] Add OAUTH2_SCOPE variable for scope configuration

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com>
Cc: Nico Rikken <nico(dot)rikken(at)alliander(dot)com>, "pgadmin-hackers(at)lists(dot)postgresql(dot)org" <pgadmin-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [pgAdmin][PATCH] Add OAUTH2_SCOPE variable for scope configuration
Date: 2021-08-31 09:07:12
Message-ID: CANxoLDfmunvjzKaZ-Vw+JGMDZxUjLAa5=ugFxPGrBiVDP5k7dA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Thanks, the patch applied.

On Tue, Aug 31, 2021 at 2:01 PM Khushboo Vashi <
khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:

>
>
> On Tue, Aug 31, 2021 at 1:25 PM Nico Rikken <nico(dot)rikken(at)alliander(dot)com>
> wrote:
>
>> Thanks for reviewing. Indeed this is a bandaid to expand the usability of
>> the OAuth implementation. As I've written on the redmine issue as well
>> https://redmine.postgresql.org/issues/6627 I've tested my usecase with a
>> modified scope by patching the oauth2.py file in production. That works
>> well for authentication from Microsoft Azure AD. Also Shy N reported this
>> method to work.
>
> Good to hear that.
>
>> This patch just makes the scope configurable so it can be adopted to
>> different use-cases.
>
> Thanks for working on this. :)
>

--
*Thanks & Regards*
*Akshay Joshi*
*pgAdmin Hacker | Principal Software Architect*
*EDB Postgres <http://edbpostgres.com>*

*Mobile: +91 976-788-8246*

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Nikhil Mohite 2021-08-31 10:43:13 Re: [pgAdmin][RM-4567]: Privileges revoked using SQL query on objects like tables does not correctly in SQL tab
Previous Message Akshay Joshi 2021-08-31 09:06:19 pgAdmin 4 commit: Introduced OAUTH2_SCOPE variable for the Oauth2 scope