MichaelRBlack opened a new pull request, #64345:
URL: https://github.com/apache/airflow/pull/64345
## Summary
- Remove the `Connection schemes (type: %s) shall not contain '_' according
to RFC3986` warning from `Connection.get_uri()` in the Task SDK
## Motivation
Standard Airflow connection types like `google_cloud_platform`,
`ssh_tunnel`, and others inherently contain underscores as defined by their
respective providers. The current warning fires on every `get_uri()` call for
these standard types, producing noisy logs that users cannot resolve since:
1. The `conn_type` values are defined by providers, not by users
2. `get_uri()` already correctly handles this by converting underscores to
dashes on the next line: `self.conn_type.lower().replace('_', '-')`
3. When using Secrets Manager backends (AWS, GCP, etc.), this warning fires
on every connection retrieval during task execution
The underscore-to-dash conversion in the URI output remains intact, ensuring
the generated URI is RFC3986 compliant.
## Test plan
- [ ] Existing `test_get_uri` test continues to pass (uses `mysql` type, no
underscores)
- [ ] Verify connection types with underscores (e.g.,
`google_cloud_platform`) no longer produce spurious warnings
- [ ] Verify `get_uri()` still produces valid RFC3986 URIs with dashes in
the scheme
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
