Skip to content

Add charset-Parameter to content-type in coupled services route - #19

Open
seitenbau-govdata wants to merge 1 commit into
OpenNRW:masterfrom
GovDataOfficial:add-charset-parameter-to-content-type-in-coupled-services-route
Open

Add charset-Parameter to content-type in coupled services route#19
seitenbau-govdata wants to merge 1 commit into
OpenNRW:masterfrom
GovDataOfficial:add-charset-parameter-to-content-type-in-coupled-services-route

Conversation

@seitenbau-govdata

@seitenbau-govdata seitenbau-govdata commented Oct 16, 2023

Copy link
Copy Markdown
Contributor

In the log file we have detected a "400 Bad Request" error entry due to an incorrect request when "reloading" the associated service metadata into the data-metadata. Distributions are often created from the service metadata and added to the metadata. If the service metadata cannot be read, distributions cannot subsequently be created from it.

The reason for the "400 Bad Request", was a URI with an umlaut in combination with the wrong encoding, ISO-8859-1 instead of UTF-8. It seems that the default behavior of the HTTP client used changed with the upgrade of Apache Camel from version 2 to version 3.

In the affected request the character "ß" in the word " Straßenbeleuchtung" is the problem.

The ISO metadata with the problematic value in "MD_Identifier" can be retrieved from the link https://gdk.gdi-de.org/inspire/srv/eng/xml_iso19139?uuid=fa00f154-4459-48d4-87b2-fcbd14e7a91f.

Steps to Reproduce:

a) Enter the CSW interface of the GDI-DE in the configuration.

db.item.csw.URL=https://gdk.gdi-de.org/gdi-de/srv/ger/csw

b) Path to retrieve metadata via Inspire bridge

/omdf/gp-csw?verb=GetRecord&identifier=fa00f154-4459-48d4-87b2-fcbd14e7a91f

This PR explicitly specifies "UTF-8" as the encoding in coupled services route for the service metadata reload query.

@seitenbau-govdata
seitenbau-govdata marked this pull request as ready for review October 20, 2023 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant