chenejac / VIVOTestMigrationJIRA

0 stars 0 forks source link

VIVO-1949: AGROVOC and LCSH concept search URLs have changed #1838

Closed chenejac closed 3 years ago

chenejac commented 3 years ago

Benjamin Gross (Migrated from VIVO-1949) said:

Agrovoc still returns a JSON result, but it says

 

{
   "semanticServicesError":{
      "message":"com.fasterxml.jackson.core.JsonParseException: Unexpected character ('<' (code 60)): expected a valid value (number, String, array, object, 'true', 'false' or 'null')\n at [Source: <!DOCTYPE HTML PUBLIC \"-//IETF//DTD HTML 2.0//EN\"><html><head><title>302 Found</title></head><body><h1>Found</h1><p>The document has moved <a href=\"https://agrovoc.uniroma2.it/agrovoc/rest/v1/search?query=tornadoes*&amp;lang=en\">here</a>.</p></body></html>; line: 1, column: 2]",
      "exception":"Exception encountered ",
      "severity":"fatal"
   },
   "conceptList":null
}

This is hopefully a quick fix... confirmed the new URL works.

chenejac commented 3 years ago

Benjamin Gross said:

https://github.com/vivo-project/VIVO/pull/201

chenejac commented 3 years ago

Benjamin Gross said:

Extended PR to include a fix for the LCSH service, as well.

chenejac commented 3 years ago

Andrew Woods said:

Looks good.

chenejac commented 3 years ago

Andrew Woods said:

See comment based on recent re-targeting to sprint-i18n: https://github.com/vivo-project/VIVO/pull/201#issuecomment-773344601

chenejac commented 3 years ago

Andrew Woods said:

Resolved with: https://github.com/vivo-project/VIVO/commit/69f698841e14f8bc2dc24f4a8034020ce1e25c43