kibana icon indicating copy to clipboard operation
kibana copied to clipboard

Fix Bulk Get User Profiles API to properly serialize parameters on the browser side.

Open azasypkin opened this issue 3 years ago • 2 comments

Summary

Before serializing Set with UIDs we should convert it to the array, otherwise it'd be serialized as {}. I also added a functional tests to make sure we'd don't break this in the future.

Flaky Test Runner: :heavy_check_mark: (x50 runs)

azasypkin avatar Aug 04 '22 09:08 azasypkin

Pinging @elastic/kibana-security (Team:Security)

elasticmachine avatar Aug 04 '22 11:08 elasticmachine

Superseded by https://github.com/elastic/kibana/pull/138361.

azasypkin avatar Aug 11 '22 08:08 azasypkin

:broken_heart: Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #39 / security app - user profiles User Profiles client side APIs "before all" hook for "can retrieve own user profile and user profiles for other users"
  • [job] [logs] FTR Configs #39 / security app - user profiles User Profiles client side APIs "before all" hook for "can retrieve own user profile and user profiles for other users"

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
security 52.1KB 52.1KB +22.0B

History

  • :green_heart: Build #64143 succeeded 95dc1d656d9c0f810cd49f01973ac2b76f935983
  • :green_heart: Build #63834 succeeded bb0e00050cdc4fb8c8208eb7ab5c899ff28e31e4
  • :green_heart: Build #63821 succeeded 770c1dabaf45cc01f600006d1ca27ee5206cda04
  • :green_heart: Build #63481 succeeded 22c7e50f91748b198b53ab09064e4f402389e63f

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

kibana-ci avatar Aug 11 '22 08:08 kibana-ci