fix: well_known_url field deprecation warning

[jacob-williamson]

I've been using the OIDCConfig class in the queue service, and ran into a bug where my tests were failing because of a deprecation warning.

def _well_known_url(self) -> str:
        if self.issuer:
            if self.well_known_url:
                LOGGER.warning(
                    DeprecationWarning(
                        "well_known_url and issuer are both set. "
                        "Defaulting to issuer URL"
                    ),
                )
            return self.issuer + "/.well-known/openid-configuration"
        return cast(str, self.well_known_url)

The issue is that even if you don't use the deprecated field (well_known_url), the model itself accesses it when it does if self.well_known_url:. This causes pydantic to emit a warning which causes tests to fail. I've replaced this with a check that doesn't access the actual field which fixes the issue.

Since pydantic also emits its own warning, we could probably get rid of the check all together.

[tpoliaw]

I think you're right - if pydantic intercepts the field access to raise a deprecation warning that is probably enough and we can remove this entire check

[jacob-williamson]

From looking at the stdio output in the tests, I now think when using the cli you don't see the pydantic warning, so may still be good to have the check

[tpoliaw]

I think this should either raise a warning or log a message. Logging a warning is not really doing either how they're intended.

[jacob-williamson]

Makes sense, I've made it a log message

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.96%. Comparing base (872ea21) to head (b223a7a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1625   +/-   ##
=======================================
  Coverage   95.96%   95.96%           
=======================================
  Files          45       45           
  Lines        3317     3317           
=======================================
  Hits         3183     3183           
  Misses        134      134           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.