Skip to content

[7.x] [Security Solutions][Detection Engine] Fixes "undefined" crash for author field by adding a migration for it (#107230)#108219

Closed
FrankHassanabad wants to merge 1 commit intoelastic:7.xfrom
FrankHassanabad:backport/7.x/pr-107230
Closed

[7.x] [Security Solutions][Detection Engine] Fixes "undefined" crash for author field by adding a migration for it (#107230)#108219
FrankHassanabad wants to merge 1 commit intoelastic:7.xfrom
FrankHassanabad:backport/7.x/pr-107230

Conversation

@FrankHassanabad
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

…thor field by adding a migration for it (elastic#107230)

## Summary

Fixes elastic#106233

During an earlier upgrade/fix to our system to add defaults to our types, we overlooked the "author" field which wasn't part of the original rules. Users upgrading might get errors such as:

```
params invalid: Invalid value "undefined" supplied to "author"
```

This fixes that issue by adding a migration for the `author` field for `7.14.1`.

See elastic#106233 for test instructions or manually remove your author field before upgrading your release and then upgrade and this should be fixed on upgrade.


### Checklist

- [x] [Unit or functional tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html) were updated or added to match the most common scenarios
@FrankHassanabad FrankHassanabad added the backport This PR is a backport of another PR label Aug 11, 2021
@FrankHassanabad FrankHassanabad enabled auto-merge (squash) August 11, 2021 15:14
@FrankHassanabad FrankHassanabad self-assigned this Aug 11, 2021
@FrankHassanabad
Copy link
Copy Markdown
Contributor Author

Closed since auto-backport ran here: #108218

auto-merge was automatically disabled August 11, 2021 15:24

Pull request was closed

@FrankHassanabad FrankHassanabad deleted the backport/7.x/pr-107230 branch August 11, 2021 15:24
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @FrankHassanabad

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants