Password Encoding and SSL Preference

[SteveDMurphy]

Closes FIDES-611

Description Of Changes

Some discovered Redshift connection issues were found to be related to password character issues along with ssl failures. This PR should make an allowance for both of those

Code Changes

• url encode database integration passwords for connection strings
• set ssl mode to prefer for all db connection types

Steps to Confirm

• Tested live with failing Redshift integration

Pre-Merge Checklist

• All CI Pipelines Succeeded
• Documentation:
  • documentation complete, PR opened in fidesdocs
  • documentation issue created in fidesdocs
• Issue Requirements are Met
• Relevant Follow-Up Issues Created
• Update CHANGELOG.md
• For API changes, the Postman collection has been updated

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	⬜️ Ignored (Inspect)	Visit Preview		Apr 12, 2024 1:35pm

[cypress]

Passing run #7228 ↗︎

0	4	0	0	0
⚠️ You've recorded test results over your free plan limit. Upgrade your plan to view test results.

Details:

Merge cddab15 into 52c6b96...
Project: fides	Commit: a84d39656f ℹ️
Status: Passed	Duration: 00:33 💡
Started: Apr 12, 2024 1:46 PM	Ended: Apr 12, 2024 1:46 PM

Review all test suite changes for PR #4790 ↗︎

[codecov]

Codecov Report

Attention: Patch coverage is 25.00000% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 86.27%. Comparing base (52c6b96) to head (d2ba11e).

Files	Patch %	Lines
src/fides/api/service/connectors/sql_connector.py	25.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4790      +/-   ##
==========================================
- Coverage   86.28%   86.27%   -0.01%     
==========================================
  Files         339      339              
  Lines       20088    20090       +2     
  Branches     2586     2586              
==========================================
  Hits        17333    17333              
- Misses       2289     2291       +2     
  Partials      466      466              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[NevilleS]

Nice 👍

[NevilleS]

The failing test here is also happening on our main branch, pretty sure from this: #4788. We'll address that separately. Pulling this in now 👍