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

          Sitaram.Vemulapalli Sitaram Vemulapalli created issue -
          yuvraj.kanwar Yuvraj Kanwar made changes -
          Field Original Value New Value
          Link This issue blocks CBSE-9383 [ CBSE-9383 ]
          isha Isha Kandaswamy made changes -
          Assignee Kamini Jagtiani [ kamini.jagtiani ] Isha Kandaswamy [ isha ]
          Sitaram.Vemulapalli Sitaram Vemulapalli made changes -
          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)



          {code:java}
          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
                  }
              ]
          }
          {code}

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

           
          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)



          {code:java}
          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
                  }
              ]
          }
          {code}

          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

           
          isha Isha Kandaswamy made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-7.0.0-4257 contains query commit c09a765 with commit message:
          MB-43384: Setting up SSL correctly

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-4257 contains query commit c09a765 with commit message: MB-43384 : Setting up SSL correctly
          isha Isha Kandaswamy made changes -
          Status In Progress [ 3 ] Open [ 1 ]
          isha Isha Kandaswamy made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          roi.katz Roi Katz made changes -
          Link This issue blocks CBSE-9563 [ CBSE-9563 ]
          Sitaram.Vemulapalli Sitaram Vemulapalli made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Sitaram.Vemulapalli Sitaram Vemulapalli made changes -
          Affects Version/s 6.6.1 [ 17002 ]
          Sitaram.Vemulapalli Sitaram Vemulapalli made changes -
          Fix Version/s 6.6.2 [ 17103 ]
          Sitaram.Vemulapalli Sitaram Vemulapalli made changes -
          Labels 6.6.2-candidate
          wayne Wayne Siu made changes -
          Link This issue blocks MB-43310 [ MB-43310 ]
          wayne Wayne Siu made changes -
          Labels 6.6.2-candidate approved-for-6.6.2
          Hide
          ajay.bhullar Ajay Bhullar added a comment -

          in 7.0.0-4373, using the repo above in my previous comment, after removing the file and killing cbq, when connected to 8093, we can still run queries and do not get the connection security config error

          Show
          ajay.bhullar Ajay Bhullar added a comment - in 7.0.0-4373, using the repo above in my previous comment, after removing the file and killing cbq, when connected to 8093, we can still run queries and do not get the connection security config error
          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-6.6.2-9484 contains query commit 106c76b with commit message:
          MB-43384: Setting up SSL correctly

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.2-9484 contains query commit 106c76b with commit message: MB-43384 : Setting up SSL correctly
          wayne Wayne Siu made changes -
          Link This issue blocks MB-42583 [ MB-42583 ]
          wayne Wayne Siu made changes -
          Labels approved-for-6.6.2 approved-for-6.5.2 approved-for-6.6.2
          wayne Wayne Siu made changes -
          Fix Version/s 6.5.2 [ 17223 ]
          Hide
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-6.5.2-6626 contains query commit 3497e00 with commit message:
          MB-43384: Setting up SSL correctly

          Show
          build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.2-6626 contains query commit 3497e00 with commit message: MB-43384 : Setting up SSL correctly
          isha Isha Kandaswamy made changes -
          Resolution Fixed [ 1 ]
          Status Reopened [ 4 ] Resolved [ 5 ]
          Hide
          ajay.bhullar Ajay Bhullar added a comment -

          verified in 6.5.2-6626, queries run correctly when connected to port 8093 and we do not get the connection security config error

          Show
          ajay.bhullar Ajay Bhullar added a comment - verified in 6.5.2-6626, queries run correctly when connected to port 8093 and we do not get the connection security config error
          Hide
          ajay.bhullar Ajay Bhullar added a comment -

          verified in 6.6.2-9484 queries run correctly when connected to port 8093 and we do not get the connection security config error after deleting the .pem file

          Show
          ajay.bhullar Ajay Bhullar added a comment - verified in 6.6.2-9484 queries run correctly when connected to port 8093 and we do not get the connection security config error after deleting the .pem file
          ajay.bhullar Ajay Bhullar made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          arunkumar Arunkumar Senthilnathan made changes -
          Labels approved-for-6.5.2 approved-for-6.6.2 approved-for-6.5.2 approved-for-6.6.2 releasenote
          lynn.straus Lynn Straus made changes -
          Fix Version/s 7.0.0 [ 17233 ]
          lynn.straus Lynn Straus made changes -
          Fix Version/s Cheshire-Cat [ 15915 ]

            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