-
Notifications
You must be signed in to change notification settings - Fork 96
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
Sympa tries creating temporary views in PostgreSQL databases unnecessarily #1812
Labels
Comments
Hi @woodfighter , Could you please apply this patch and check if the problem will be solved? |
Yes, that seems to do the trick. |
ikedas
added a commit
that referenced
this issue
Sep 6, 2024
PostgreSQL/SQLite: Sympa tries creating temporary views in databases unnecessarily (#1812)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version
Sympa 6.2.72
Installation method
Freebsd Port
Expected behavior
Users for databases which are included as a data source for lists should be able to have minimal privileges.
Actual behavior
On every connect to a PostgreSQL database Sympa's database driver tries to create a temporary view named dual (for compatibility with Oracle) since baed879 . For the Sympa's installations user that is a tiny bit inefficient but it probably already has enough privileges anyway, but if you're including separate databases as data sources then you have to give the user the TEMPORARY privilege even though only the SELECT privilege is needed for the query itself.
Steps to reproduce
Additional information
It's a minor issue and we're currently working around it by granting the necessary privileges. Perhaps someone sees a way to rewrite the SQL statements (as far as I see all usages of dual are look like this one:
sympa/src/lib/Sympa/Family.pm
Line 521 in a9dfb33
The text was updated successfully, but these errors were encountered: