Skip to content

Change the Authentication mechanism for fleet.#14867

Merged
ph merged 3 commits intoelastic:fleetfrom
ph:agent/change-auth-mechanism
Dec 2, 2019
Merged

Change the Authentication mechanism for fleet.#14867
ph merged 3 commits intoelastic:fleetfrom
ph:agent/change-auth-mechanism

Conversation

@ph
Copy link
Copy Markdown
Contributor

@ph ph commented Nov 29, 2019

Fleet changed how we should authenticate with them instead of using an
api key and a special header fields we now use the normal "Authorization" key and with this format:

Authorization: ApiKey {accessApiKey}

See elastic/kibana#49639 for details

Fleet changed how we should authenticate with them instead of using an
api key and a special header fields we now use the normal "Authorization" key and with this format:

```
Authorization: ApiKey {accessApiKey}
```

See #49639 for details
Comment thread x-pack/agent/pkg/fleetapi/round_trippers.go Outdated
@ph ph requested a review from michalpristas November 29, 2019 19:28
@ph ph self-assigned this Nov 29, 2019
@michalpristas
Copy link
Copy Markdown
Contributor

no changes needed on enroll part?

@ph
Copy link
Copy Markdown
Contributor Author

ph commented Dec 2, 2019

@michalpristas Yes, it's done in another PR in #14869

Copy link
Copy Markdown
Contributor

@michalpristas michalpristas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@ph ph merged commit 5d49777 into elastic:fleet Dec 2, 2019
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
* Change the Authentication mechanism for fleet.

Fleet changed how we should authenticate with them instead of using an
api key and a special header fields we now use the normal "Authorization" key and with this format:

```
Authorization: ApiKey {accessApiKey}
```

See elastic#49639 for details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants