Skip to content

Remove guidance to lowercase http.request.method.#1443

Merged
ebeahan merged 4 commits intoelastic:masterfrom
ebeahan:remove/http.request.method-lowercase-guidance
Jun 2, 2021
Merged

Remove guidance to lowercase http.request.method.#1443
ebeahan merged 4 commits intoelastic:masterfrom
ebeahan:remove/http.request.method-lowercase-guidance

Conversation

@ebeahan
Copy link
Copy Markdown
Member

@ebeahan ebeahan commented May 28, 2021

In addition to removing the deprecation note, I included new guidance about retaining the casing from the original event.

I also limited the Example attribute to a single value since this field is not expected to be an array.

Closes #1363

@ebeahan ebeahan self-assigned this May 28, 2021
@ebeahan ebeahan added the 8.0.0 label May 28, 2021
@ebeahan ebeahan requested a review from a team May 28, 2021 22:11
Copy link
Copy Markdown
Contributor

@djptek djptek left a comment

Choose a reason for hiding this comment

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

LGTM

@ebeahan ebeahan merged commit f6d659c into elastic:master Jun 2, 2021
@ebeahan ebeahan deleted the remove/http.request.method-lowercase-guidance branch June 2, 2021 17:38
@epixa epixa added the breaking label Jun 7, 2021
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.

Remove guidance to lowercase http.request.method

3 participants