Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: Cheshire-Cat, 6.6.1
-
Component/s: query
-
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
Activity
| Field | Original Value | New Value |
|---|---|---|
| Link | This issue blocks CBSE-9383 [ CBSE-9383 ] |
| Assignee | Kamini Jagtiani [ kamini.jagtiani ] | Isha Kandaswamy [ isha ] |
| 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 |
| Status | Open [ 1 ] | In Progress [ 3 ] |
| Status | In Progress [ 3 ] | Open [ 1 ] |
| Resolution | Fixed [ 1 ] | |
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Link | This issue blocks CBSE-9563 [ CBSE-9563 ] |
| Resolution | Fixed [ 1 ] | |
| Status | Resolved [ 5 ] | Reopened [ 4 ] |
| Affects Version/s | 6.6.1 [ 17002 ] |
| Fix Version/s | 6.6.2 [ 17103 ] |
| Labels | 6.6.2-candidate |
| Link | This issue blocks MB-43310 [ MB-43310 ] |
| Labels | 6.6.2-candidate | approved-for-6.6.2 |
| Link | This issue blocks MB-42583 [ MB-42583 ] |
| Labels | approved-for-6.6.2 | approved-for-6.5.2 approved-for-6.6.2 |
| Fix Version/s | 6.5.2 [ 17223 ] |
| Resolution | Fixed [ 1 ] | |
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
| Labels | approved-for-6.5.2 approved-for-6.6.2 | approved-for-6.5.2 approved-for-6.6.2 releasenote |
| Fix Version/s | 7.0.0 [ 17233 ] |
| Fix Version/s | Cheshire-Cat [ 15915 ] |