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

setupSSL fails at start none of the queries work

    XMLWordPrintable

    Details

    • Triage:
      Untriaged
    • Story Points:
      1
    • Is this a Regression?:
      Unknown

      Description

      The following need to be investigated and see if it is right

      newKeyspace() is called all queries (non ssl queries too)

      https://github.com/couchbase/query/blob/master/datastore/couchbase/couchbase.go#L1265

      Assume p.store.connSecConfig is nil none of the queries works.

      Where do we set this https://github.com/couchbase/query/blob/master/server/http/service_endpoint.go#L379
      This is setupSSL() Assume if the function bailout before reaching what happens. To prove that add return nil or error https://github.com/couchbase/query/blob/master/server/http/service_endpoint.go#L340
      and start server and execute any query (non-ssl setup)

      select * from default;
      {
          "requestID": "23c53059-9466-497b-a166-ef4996daaa7c",
          "errors": [
              {
                  "code": 12023,
                  "msg": "Connection security config not provided. Unable to load bucket default:default",
                  "retry": true
              }
          ]
      }
      

      Also following message must be changed to Infof not Errorf
      https://github.com/couchbase/query/blob/master/server/http/service_endpoint.go#L370

      Also do you need posting remote query service ? https://github.com/couchbase/query/blob/master/server/http/service_endpoint.go#L387

        Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

          No work has yet been logged on this issue.

            People

            Assignee:
            isha Isha Kandaswamy
            Reporter:
            Sitaram.Vemulapalli Sitaram Vemulapalli
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Gerrit Reviews

                There are no open Gerrit changes

                  PagerDuty