[confignet] Change Transport from string to TransportType

[TylerHelmuth]

Description:
Changes Transport from a string to a new TransportType. Implements UnmarshalText for TransportType to enforce values.

This PR may be too much - it introduces a breaking change a lot of new public APIs that may not be worth it for such a small module. If we don't like the surface area this creates or the breaking change, but we still want to enforce transport type values, I think implementing Validate keeps the API footprint smaller and isn't breaking.

Link to tracking Issue:

Closes #9364

Documentation:

Added godoc comments

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.97%. Comparing base (cc485e0) to head (21608ea).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #9385   +/-   ##
=======================================
  Coverage   90.96%   90.97%           
=======================================
  Files         353      353           
  Lines       18626    18640   +14     
=======================================
+ Hits        16943    16957   +14     
  Misses       1356     1356           
  Partials      327      327           

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

[mx-psi]

IMO this PR should not make any changes that affect end users like changing the struct tags related to this.

We may consider doing the breaking change on a separate PR (personally I don't think it is worth it), but since we can consider the changes separately, I think we should keep each change on a separate PR

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[mx-psi]

@bogdandrutu This is waiting on your review

[TylerHelmuth]

All tests are now passing except Contrib, which I'll fixed after this is merged should we choose to go forward with this breaking change approach.