Omit field from mapping payload if it matches index field; Abort detector creation when mapping has failed

[amsiglan]

Description

In this PR we have updated two behaviors:

1. A fix was made as part of return rule fields which do not have aliases security-analytics#652 where rule fields that match with index fields are now shown as part of automatic mappings in the view mappings API. However, a field cannot be mapped backed to itself in an index i.e. an alias name cannot be same as the index field name, so as part of create mapping API payload we should omit the automatically mapped field that matched an index field.
2. If create mapping fails when user hits Create Detector, then we should abort the creation altogether. This behavior was regressed in an earlier version. This PR fixes that.

Check List

• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov]

Codecov Report

Merging #752 (352cfa1) into main (b581e58) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #752      +/-   ##
==========================================
- Coverage   31.28%   31.28%   -0.01%     
==========================================
  Files         150      150              
  Lines        4913     4920       +7     
  Branches      880      882       +2     
==========================================
+ Hits         1537     1539       +2     
- Misses       3192     3197       +5     
  Partials      184      184              

see 2 files with indirect coverage changes