Added setting for python-specific connection URL

[Floris-Hyve]

Problem:

Standard database connection string does not work when using MySQL python connector for migration. For example, ssl connection is disabled for the standard connection URL, but ssl connection is required for python MySQL connection.

Fix:

Added Python-specific connection URL, and implemented it in the get_database_properties function, so the correct URL is selected for the migration script.

Input URL is given in the portal.properties:

db.connection_string_python=

[pvannierop]

Some small changes.

[pvannierop]

Two proposed changes:

1. Clean up the commits (now there are 7!)
2. Harmonize information on precedence of the URL properties.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

[JREastonMarks]

From an RFC72 (6.0) perspective the core module has been moved out of the cbioportal code base and into its own code base here: https://github.com/cBioPortal/cbioportal-core

@pvannierop can you create another PR for this in that repo? That way it will work in both 5.X and 6.X versions of cBioPortal.

[alisman]

@JREastonMarks can you provide direction on how re-target this PR in a 6.0 world.

[JREastonMarks]

Hi @Floris-Hyve,
With the release of 6.0.0 (RFC72) we moved a lot of code around and updated some of the files that you reference in this PR. The core module has been moved to its own repository: https://github.com/cBioPortal/cbioportal-core. If you move this PR to the core package it should still be a quick change. Please let me know if you need any help.

[JREastonMarks]

Closing but willing to reopen if needed.