HA-446 - DatahubConnector was not using token when configuring the ConnectionConfig/DatahubSchema

[andres-torres-marroquin]

Closes HA-446

Description Of Changes

DatahubConnector was not using token when configuring the ConnectionConfig/DatahubSchema

Code Changes

• Updates to DatahubConnector

Steps to Confirm

1. Configure a Datahub integration that requires token.

Pre-Merge Checklist

• Issue requirements met
• All CI pipelines succeeded
• CHANGELOG.md updated
  • Add a db-migration This indicates that a change includes a database migration label to the entry if your change includes a DB migration
  • Add a high-risk This issue suggests changes that have a high-probability of breaking existing code label to the entry if your change includes a high-risk change (i.e. potential for performance impact or unexpected regression) that should be flagged
• Followup issues:
  • Followup issues created (include link)
  • No followup issues
• Database migrations:
  • Ensure that your downrev is up to date with the latest revision on main
  • Ensure that your downgrade() migration is correct and works
    • If a downgrade migration is not possible for this change, please call this out in the PR description!
  • No migrations
• Documentation:
  • Documentation complete, PR opened in fidesdocs
  • Documentation issue created in fidesdocs
  • If there are any new client scopes created as part of the pull request, remember to update public-facing documentation that references our scope registry
  • No documentation updates required

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
fides-plus-nightly	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 14, 2025 4:54pm

[cypress]

fides    Run #12482

Run Properties:   Passed #12482  •  b4dc5bb62f: HA-446 - DatahubConnector was not using token when configuring the ConnectionCon...

Project	fides
Branch Review	main
Run status	Passed #12482
Run duration	00m 49s
Commit	b4dc5bb62f: HA-446 - DatahubConnector was not using token when configuring the ConnectionCon...
Committer	Andres Torres
View all properties for this run ↗︎

---

Test results
Failures	0
Flaky	0
Pending	0
Skipped	0
Passing	5
⚠️ You've recorded test results over your free plan limit. Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.20%. Comparing base (a337457) to head (e307df8).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #5762   +/-   ##
=======================================
  Coverage   87.19%   87.20%           
=======================================
  Files         391      391           
  Lines       24312    24313    +1     
  Branches     2632     2632           
=======================================
+ Hits        21200    21201    +1     
  Misses       2537     2537           
  Partials      575      575           

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