Uploaded image for project: 'REST Client'
  1. REST Client
  2. MGNLRESTCL-172

The countries rest client in declarative-rest-demo no longer works

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 2.0.2
    • 2.0.1
    • None
    • DevX 1, DevX 2, DevX 3
    • 8

      https://git.magnolia-cms.com/projects/MODULES/repos/rest-client/browse/declarative-rest-demo/restClients/countries.yaml

      The API has changed (new domain owner?) and is probably behind an API key wall:

      (https://countrylayer.com/documentation/)
      https://api.countrylayer.com/v2/
      Opening "countries" in the mgnl Rest Client app produces a "Bad request."

       

       
      2021-11-11 08:18:57,224 TRACE class org.ehcache.core.Ehcache-REST_CLIENT_CACHE  : Initializing
      2021-11-11 08:18:57,227 DEBUG class org.ehcache.core.Ehcache-REST_CLIENT_CACHE  : Initialize successful.
      2021-11-11 08:23:19,560 ERROR info.magnolia.rest.client.timeout.TimeoutHandler  : Timeout error while issuing a rest call, cause by: RESTEASY004655: Unable to invoke request: org.apache.http.conn.HttpHostConnectException: Connect to restcountries.eu:443 [restcountries.eu/15.197.142.173, restcountries.eu/3.33.152.147] failed: Connection timed out (Connection timed out)
      2021-11-11 08:23:19,561 WARN  t.client.exception.handler.DefaultExceptionHandler: Error while issuing a rest call, target: https://restcountries.eu/rest/v2/all, cause by: RESTEASY004655: Unable to invoke request: org.apache.http.conn.HttpHostConnectException: Connect to restcountries.eu:443 [restcountries.eu/15.197.142.173, restcountries.eu/3.33.152.147] failed: Connection timed out (Connection timed out), cause by: Connect to restcountries.eu:443 [restcountries.eu/15.197.142.173, restcountries.eu/3.33.152.147] failed: Connection timed out (Connection timed out)
      

       

      Might also need an update:

      https://git.magnolia-cms.com/projects/MODULES/repos/rest-client/browse/declarative-rest-demo/templates/components/countries/countries.ftl

       Add integration tests to the CE project to verify demo is working.

       

       Demo also references old mtk instead of mtk2.

       

        Acceptance criteria

              canh.nguyen Canh Nguyen
              mdrapela Martin Drápela
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved:

                  Bug DoR
                  Task DoD

                    Estimated:
                    Original Estimate - Not Specified
                    Not Specified
                    Remaining:
                    Remaining Estimate - Not Specified
                    Not Specified
                    Logged:
                    Time Spent - 6.25d
                    6.25d