WCS: ensure identifier is a string (not a list) (#1010)

[tomkralidis]

Fixes #1010

[tomkralidis]

FYI CI failures are unrelated to this PR.

[geographika]

Thanks @tomkralidis.

Looks like CI is failing due to (unrelated) http://34.67.197.57:8585 being offline. Used in

OWSLib/tests/test_ogcapi_connectedsystems_osh.py

Lines 22 to 23 in 7c4ca6f

	NODE_TEST_OK_URL = 'http://34.67.197.57:8585/sensorhub/test'
	# Directs to OSH hosted test server

Was using a list for the identifier an error (I presume so as it is no longer supported)?

[tomkralidis]

Was using a list for the identifier an error (I presume so as it is no longer supported)?

It was for downstream applications passing identifier as a str in getCoverage. The PR removes the ambiguity.

[geographika]

Was using a list for the identifier an error (I presume so as it is no longer supported)?

It was for downstream applications passing identifier as a str in getCoverage. The PR removes the ambiguity.

Ok. +1 for merging, but as this will break existing user code, I guess we should bump the version to 0.36.0.

[tomkralidis]

Ok. +1 for merging, but as this will break existing user code, I guess we should bump the version to 0.36.0.

Yes, +1 on next release.

[tomkralidis]

Looks like CI is failing due to (unrelated) http://34.67.197.57:8585 being offline. Used in

OWSLib/tests/test_ogcapi_connectedsystems_osh.py

Lines 22 to 23 in 7c4ca6f

	NODE_TEST_OK_URL = 'http://34.67.197.57:8585/sensorhub/test'
	# Directs to OSH hosted test server

Dedicated issue opened in #1013.