Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-22515

N1QL CURL should set the User Agent String

    XMLWordPrintable

Details

    Description

      Take these three example requests:

      $ curl 10.111.95.102:8091/pools/default


      $ http 10.111.95.102:8091/pools/default


      cbq> select CURL("GET", "10.111.95.102:8091/pools/default");

      which produce the following (respectively) in http_access.log:

      10.111.95.1 - - [27/Jan/2017:21:40:27 +0000] "GET /pools/default HTTP/1.1" 200 3742 - curl/7.49.1
      		 
      10.111.95.1 - - [27/Jan/2017:21:40:33 +0000] "GET /pools/default HTTP/1.1" 200 3742 - HTTPie/0.9.3
      		 
      10.111.95.1 - - [27/Jan/2017:21:44:15 +0000] "GET /pools/default HTTP/1.1" 200 3757 - -

      Obviously, this isn't dependant on the receiving end being CB, but it demonstrates the point. I suspect that we would want this from a best practice perspective anyway, but it's going to be vital for debugging issues where N1QL CURL is being aimed at a CB cluster.

      Attachments

        Issue Links

          relates to

          Task - A task that needs to be done. MB-21792 Add function DATASTORE_VERSION() / DS_VERSION()

          • Major - Major loss of function.
          • Closed

          Task - A task that needs to be done. MB-22526 N1QL CURL should explicitly state the CB version in User Agent

          • Critical - Crashes, loss of data, severe memory leak.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. MB-23133 CURL : Create custom http header for N1QL CURL to allow localhost endpoints to enable/disable access

          • Major - Major loss of function.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              ajay.bhullar Ajay Bhullar
              James Flather James Flather (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty