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

Superuser Login required when using an Authentication Configuration #905

Open
webrian opened this issue May 1, 2024 · 4 comments
Open
Labels

Comments

@webrian
Copy link
Contributor

webrian commented May 1, 2024

Is there a specific reason why a superuser login is required when using an authentication configuration?

When using a user name and password in Basic Authentication, however, no superuser login is required.

Bildschirmfoto vom 2024-05-01 10-11-52

@signedav
Copy link
Member

signedav commented May 6, 2024

In case you have a normal login (without superuser) defined in a generally named authentication (maybe something others have as well) and then you create the schema and you need superuser login for that, then you could edit your own authentication file, yes, but it would be easier to keep it untouched and just create it with the superuser settings from the Model Baker settings. Or do I understand it wrong?

@webrian
Copy link
Contributor Author

webrian commented May 6, 2024

No, this is about the data export workflow.

Suppose I connect to the database with a user with read permission (SELECT privileges) only.

Case A:
I connect to the database using an authentication configuration (in Configuration tab):

Bildschirmfoto vom 2024-05-01 10-11-52

The wizard forces me to check Execute data management tasks with superuser login from settings (postgres), but 1. as a "normal" user I do not have superuser access to our production database 2. (and more important) it should not be necessary for an export, as no table or schema is changed. Thus no export can be done.

Whereas case B:
I connect to the database using Basic auth with username and password (in Basic tab):

There is no need to check Execute ... with superuser login ... and it works anyway as expected:

Bildschirmfoto vom 2024-05-06 11-21-33

Bildschirmfoto vom 2024-05-06 11-21-56

I don't see any reason why a superuser login is required when using the authentication configuration, but not when entering username and password. On our production database only a few users have more than read privileges on certain selected schemas.

@signedav
Copy link
Member

signedav commented May 6, 2024

Ah now I get it. I haven't seen the message below and assumed you want to ask, why there is the option to use superuser...

Well:

  1. yes, on export this option could be hidden, but this is not your issue here anyway.

  2. Why it requires superuser login on authentication configuration I don't know. I'll have to check that...

@signedav signedav added the bug label May 6, 2024
@signedav
Copy link
Member

signedav commented May 6, 2024

(not sure if it's a bug, but I added it to have it on the list)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants