Skip to content

Partition service feature flag impl fix

Alok Joshi requested to merge partition_ff_fix into master

Partition service apis only work with a service principal token, returns 403 with user tokens. This is expected behavior

In the feature flag implementation (in case of data partition strategy), we must generate a service principal token and use it as authorization header instead of default token from original request.

Another smaller change in this MR is to remove the feature flag config bean and instead make the impl classes as beans themselves, as the config approach fails pre-emptively when trying to resolve beans such as DpsHeaders and IServiceAccountJwtClient

Couple of other small fixes:

  • Add missing correlation ID to the request headers
  • Log PartitionException and throw AppException instead of ignoring the exception
Edited by Alok Joshi

Merge request reports