Uploaded image for project: 'mod-inventory'
  1. mod-inventory
  2. MODINV-156

Does not pass X-Okapi-Request-Id

    XMLWordPrintable

    Details

    • Template:
    • Sprint:
      CP: sprint 72
    • Story Points:
      2
    • Development Team:
      Core: Platform

      Description

      All business modules that takes request from Okapi should pass on X-Okapi-Request-Id so that the call graph can be established.. mod-inventory does not do that.

      Example url: /inventory/items?query=barcode%3D%3D1653577489

      2019-09-13 09:45:10,626 INFO  ProxyContext         117906/inventory REQ 10.0.2.2:46652 diku GET /inventory/items mod-authtoken-2.3.0-SNAPSHOT.53 mod-inventory-13.1.0-SNAPSHOT.170
      2019-09-13 09:45:10,790 INFO  ProxyContext         117906/inventory RES 202 163730us mod-authtoken-2.3.0-SNAPSHOT.53 http://10.0.2.15:9132/inventory/items?query=barcode%3D%3D1653577489
      2019-09-13 09:45:11,154 INFO  ProxyContext         117906/inventory RES 200 363755us mod-inventory-13.1.0-SNAPSHOT.170 http://10.0.2.15:9143/inventory/items?query=barcode%3D%3D1653577489
      

      All intermediate steps omit the 117906 – the top-level request-Id.. From code inspection it appears that org.folio.inventory.support.http.client.OkapiHttpClient never passes on X-Okapi-Request-Id.

      See https://issues.folio.org/browse/CIRC-452?focusedCommentId=59979&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-59979
      for how it should look..

        TestRail: Results

          Attachments

            Issue Links

              Activity

                People

                Assignee:
                adam Adam Dickmeiss
                Reporter:
                adam Adam Dickmeiss
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                  Dates

                  Created:
                  Updated:
                  Resolved:

                    TestRail: Runs

                      TestRail: Cases