-
Notifications
You must be signed in to change notification settings - Fork 94
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
Issue 4168 - Console - RGPD "export my data" - Geonetwork on an other database #4189
base: master
Are you sure you want to change the base?
Conversation
Change datasource name to fit configuration
fwiw those |
No it needed by Spring because two datasource beans are declared in configuration. Each field must have the same field name as an object and in configuration. Otherwise application won't start |
such voodoo :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about default values ? Don't know if we can set something like ${pgsqlGNHost:pgsqlHost}
to avoid duplicating values in console.properties.
And we will need a PR in georchestra/datadir
with it.
Yes I can do this, good idea for default value
Yes this is needed, but Integration test are not working on geonetwork and I didn't had time to look. Once Integration Test is fixed I will to the datadir and ansible PR as well |
This PR add a new datasource to connect to geonetwork database.
New fields have to be added in console.properties :
PostGreSQL database connection parameters to geonetwork
pgsqlGNHost
pgsqlGNPort
pgsqlGNDatabase
pgsqlGNUser
pgsqlGNPassword
PR will be done for those new params in datadir and ansible.