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

Add sslmode to QGIS .qgs project #470

Closed
wants to merge 2 commits into from
Closed

Add sslmode to QGIS .qgs project #470

wants to merge 2 commits into from

Conversation

ponceta
Copy link
Member

@ponceta ponceta commented Oct 23, 2024

Add sslmode to QGIS .qgs project

  • sslmode=prefer as default parameter

Add sslmode to QGIS .qgs project
@ponceta ponceta self-assigned this Oct 23, 2024
Add project translation scheduler, create package when altering the project.
@urskaufmann
Copy link
Contributor

I have always used ssl mode = prefer, when adding layers to the project. But in the project was no ssl entry. Why?

@ponceta
Copy link
Member Author

ponceta commented Oct 23, 2024

I have always used ssl mode = prefer, when adding layers to the project. But in the project was no ssl entry. Why?

Probably because we use a pg_conf that can handle it directly? No clear idea why exactly, and not sure if enforcing it in the qgs file is a good idea or not. This was mainly harmonization work between TWW and TDH.

@cymed
Copy link
Contributor

cymed commented Oct 24, 2024

I am in favour of leaving it empty. See also qgis/QGIS#58936

@cymed
Copy link
Contributor

cymed commented Oct 24, 2024

Also it is smart imho to only have on open pr altering the qgs project. It is easy to overwrite changes otherwise

@ponceta
Copy link
Member Author

ponceta commented Oct 25, 2024

As discussed in upstream QGIS issue, it makes no sense to add sslmode to a pg_service configured QGIS project.

This has to be taken into account on TDH project as well.

@ponceta ponceta closed this Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants