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

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

    XMLWordPrintable

Details

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

    Description

      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.

       

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

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

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Bug DoR
                    Task DoD

                    Time Tracking

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