Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to v7.11.0 #575

Merged
merged 2 commits into from
Feb 11, 2021
Merged

Update to v7.11.0 #575

merged 2 commits into from
Feb 11, 2021

Conversation

github-actions[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

@antoineco antoineco closed this Feb 11, 2021
@antoineco antoineco reopened this Feb 11, 2021
@antoineco
Copy link
Collaborator

antoineco commented Feb 11, 2021

Something possibly changed in Enterprise Search. The CI fails with

[+] Retrieving private key from Elasticsearch

null
.github/workflows/scripts/run-tests-enterprise-search.sh: line 27: null: unbound variable
Error: Process completed with exit code 1.

I'll take a look later.


edit: found it. The index's suffix has changed from _v2 to _v3, and the API keys are now (finally) encrypted.

$ curl -sD- -u elastic:changeme 'http:https://localhost:9200/_search?q=name:private-key&pretty'

HTTP/1.1 200 OK
Warning: 299 Elasticsearch-7.11.0-8ced7813d6f16d2ef30792e2fcde3e755795ee04 "this request accesses system indices: [.security-7, .triggered_watches, .watches], but in a future major version, direct access to system indices will be prevented by default"
content-type: application/json; charset=UTF-8
content-length: 1275

{
  "took" : 24,
  "timed_out" : false,
  "_shards" : {
    "total" : 92,
    "successful" : 92,
    "skipped" : 0,
    "failed" : 0
  },
  "hits" : {
    "total" : {
      "value" : 1,
      "relation" : "eq"
    },
    "max_score" : 0.6931471,
    "hits" : [
      {
        "_index" : ".ent-search-actastic-app_search_api_tokens_v3",
        "_type" : "_doc",
        "_id" : "loco_moco_account_id:6025038252b42a46f77aeb32|name:private-key",
        "_score" : 0.6931471,
        "_source" : {
          "id" : "loco_moco_account_id:6025038252b42a46f77aeb32|name:private-key",
          "created_at" : "2021-02-11T10:14:26Z",
          "updated_at" : "2021-02-11T10:14:26Z",
          "loco_moco_account_id" : "6025038252b42a46f77aeb32",
          "authentication_token" : {
            "hash" : "e6fdada3481f795b55dc0e6fb5742f376398d0e5056607c93390703c939f3b45",
            "ciphertext" : "YnZZRXVXcWhLYTQvdWtmekROVXZ0eVN5UC9Ock9xbmJ1WmlVTCtGMTQ0bTM3TyttUWthTS9sS21hUUJ1b2F1SC0tSmx6S2RqTzQyKzRDOG5HQTluVWxUdz09--6927142cb8ee2d913da18d646a235ee0d4bedb8b"
          },
          "read_access" : true,
          "write_access" : true,
          "access_all_engines" : true,
          "name" : "private-key",
          "token_type" : "private"
        }
      }
    ]
  }
}

@antoineco antoineco force-pushed the create-pull-request/patch branch 2 times, most recently from 4c19891 to 1452f13 Compare February 11, 2021 11:33
App Search API keys are now encrypted in Elasticsearch, so we simply
ensure they exist and don't try to read their value.
@antoineco antoineco merged commit 75ebebc into main Feb 11, 2021
@antoineco antoineco deleted the create-pull-request/patch branch February 11, 2021 12:48
DanBrown47 pushed a commit to DanBrown47/docker-elk that referenced this pull request Jun 22, 2023
ci: Update Enterprise Search test for v3 index

App Search API keys are now encrypted in Elasticsearch, so we simply
ensure they exist and don't try to read their value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant