Potential attributeError in split() on connection.host

### Apache Airflow version

3.1.2

### If "Other Airflow 2/3 version" selected, which one?

_No response_

### What happened?

The `.split()` method is called on `connection.host` which may be `None`
According to Airflow's `Connection` model definition https://github.com/apache/airflow/blob/4a083b49aa67ee89ddc1cdbf82aa0babccd370a6/airflow-core/src/airflow/models/connection.py#L136 , the `host` field type is `str | None` and is `nullable=True` in the database, so `connection.host` may be `None`.
But the existing test [`test_openlineage_methods_with_redshift`](https://github.com/apache/airflow/blob/4a083b49aa67ee89ddc1cdbf82aa0babccd370a6/providers/postgres/tests/unit/postgres/hooks/test_postgres.py#L144-L175)  only tests cases where `host` is a concrete string value, but does not test `host=None`. The test allows `port` to be `None` (line 128), but the `host` parameterization list does not include none values.

### What you think should happen instead?

_No response_

### How to reproduce

https://github.com/apache/airflow/blob/4a083b49aa67ee89ddc1cdbf82aa0babccd370a6/airflow-core/src/airflow/models/connection.py#L136
https://github.com/apache/airflow/blob/4a083b49aa67ee89ddc1cdbf82aa0babccd370a6/providers/postgres/tests/unit/postgres/hooks/test_postgres.py#L144-L175


### Operating System

macOS

### Versions of Apache Airflow Providers

_No response_

### Deployment

Official Apache Airflow Helm Chart

### Deployment details

_No response_

### Anything else?

_No response_

### Are you willing to submit PR?

- [x] Yes I am willing to submit a PR!

### Code of Conduct

- [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)


	def test_openlineage_methods_with_redshift(
	self,
	mocker,
	aws_conn_id,
	port,
	host,
	conn_cluster_identifier,
	expected_host,
	):
	mock_aws_hook_class = mocker.patch("airflow.providers.amazon.aws.hooks.base_aws.AwsBaseHook")

	mock_conn_extra = {
	"iam": True,
	"redshift": True,
	}
	if aws_conn_id is not NOTSET:
	mock_conn_extra["aws_conn_id"] = aws_conn_id
	if conn_cluster_identifier is not NOTSET:
	mock_conn_extra["cluster-identifier"] = conn_cluster_identifier

	self.connection.extra = json.dumps(mock_conn_extra)
	self.connection.host = host
	self.connection.port = port

	# Mock AWS Connection
	mock_aws_hook_instance = mock_aws_hook_class.return_value
	mock_aws_hook_instance.region_name = "us-east-1"

	assert (
	self.db_hook._get_openlineage_redshift_authority_part(self.connection)
	== f"{expected_host}:{port or 5439}"
	)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Potential attributeError in split() on connection.host #58299

Apache Airflow version

If "Other Airflow 2/3 version" selected, which one?

What happened?

What you think should happen instead?

How to reproduce

Operating System

Versions of Apache Airflow Providers

Deployment

Deployment details

Anything else?

Are you willing to submit PR?

Code of Conduct

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Potential attributeError in split() on connection.host #58299

Description

Apache Airflow version

If "Other Airflow 2/3 version" selected, which one?

What happened?

What you think should happen instead?

How to reproduce

Operating System

Versions of Apache Airflow Providers

Deployment

Deployment details

Anything else?

Are you willing to submit PR?

Code of Conduct

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions