Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable master password in server mode #8299

Open
johansmitsnl opened this issue Dec 27, 2024 · 4 comments
Open

Disable master password in server mode #8299

johansmitsnl opened this issue Dec 27, 2024 · 4 comments
Assignees
Milestone

Comments

@johansmitsnl
Copy link

Describe the solution you'd like

We are using oauth2 for authentication and the idea is that we don't want to have password (besides the sql servers) to access pgadmin. There is PGADMIN_CONFIG_MASTER_PASSWORD_REQUIRED but this only works in desktop mode. Can this also support server mode?

Describe alternatives you've considered

None

@drehelis
Copy link

Upgrading from 8.11 -> 8.14 brings back "Set Master Password" popup even thought PGADMIN_CONFIG_MASTER_PASSWORD_REQUIRED was in place.

PGADMIN_CONFIG_AUTHENTICATION_SOURCES="['oauth2']" is also set for us.

@khushboovashi khushboovashi added Bug and removed Feature labels Jan 1, 2025
@anilsahoo20 anilsahoo20 moved this to 🆕 New in Current Sprint (185) Jan 1, 2025
@yogeshmahajan-1903 yogeshmahajan-1903 self-assigned this Jan 3, 2025
@yogeshmahajan-1903 yogeshmahajan-1903 moved this from 🆕 New to 🏗 In Progress in Current Sprint (185) Jan 3, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 3, 2025
@yogeshmahajan-1903 yogeshmahajan-1903 moved this from 🏗 In Progress to In Review in Current Sprint (185) Jan 3, 2025
@akshay-joshi akshay-joshi moved this from In Review to In Testing in Current Sprint (185) Jan 3, 2025
@akshay-joshi akshay-joshi added this to the 9.0 milestone Jan 3, 2025
@knutia
Copy link

knutia commented Jan 7, 2025

Same issue using :
PGADMIN_CONFIG_AUTHENTICATION_SOURCES="['webserver']"
Was working in fine in 8.13
Upgrading from 8.13 -> 8.14 stoped working.

@leo-genglfs
Copy link

Upgrading from 8.11 -> 8.14 brings back "Set Master Password" popup even thought PGADMIN_CONFIG_MASTER_PASSWORD_REQUIRED was in place.

PGADMIN_CONFIG_AUTHENTICATION_SOURCES="['oauth2']" is also set for us.

We have the same issue, upgraded from 8.13 to 8.14, then MASTER_PASSWORD_REQUIRED = False stopped working.

Getting a prompt for setup master password after Login via Azure Oauth2
Image

Our Auth is configured as:
AUTHENTICATION_SOURCES = ['oauth2']
with OAUTH2_CONFIG points to Azure, this parts works fine.

@anilsahoo20
Copy link
Contributor

@yogeshmahajan-1903 found some issues while testing,

  1. AUTHENTICATION_SOURCES = ['oauth2'], MASTER_PASSWORD_REQUIRED=False, and on Login>Open query tool of a DB>Logout, it gives the below error
{
  "success": 0,
  "errormsg": "'NoneType' object is not subscriptable",
  "info": "",
  "result": null,
  "data": null
}
  1. AUTHENTICATION_SOURCES = ['internal'] or AUTHENTICATION_SOURCES = ['oauth2'], and on Login>Open query tool of a DB>Logout, it gives the below error
{
  "success": 0,
  "errormsg": "'utf-8' codec can't decode byte 0x8b in position 0: invalid start byte",
  "info": "",
  "result": null,
  "data": null
}

@anilsahoo20 anilsahoo20 moved this from In Testing to 🏗 In Progress in Current Sprint (185) Jan 20, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 20, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 20, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 20, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 20, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 21, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 21, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 22, 2025
yogeshmahajan-1903 added a commit to yogeshmahajan-1903/pgadmin4 that referenced this issue Jan 22, 2025
@yogeshmahajan-1903 yogeshmahajan-1903 moved this from 🏗 In Progress to In Testing in Current Sprint (185) Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Testing
Development

No branches or pull requests

9 participants