[IGN-14156] Remove some unused field on the Gateway UI for an extension point

Module Development
1

Can we remove some unused field on the Gateway UI for an extension point.
I don't need retentionDays in AuditProfileConfig

perhaps just removing it in SchemaUtil.fromType(AuditProfileConfig.class) will be ok ?

    @Override
    public Optional<WebUiComponent> getWebUiComponent(ComponentType type) {
        return Optional.of(
                new ExtensionPointResourceForm(
                        AuditProfileType.RESOURCE_TYPE,
                        "AuditLogSplitterType",
                        TYPE_ID,
                        SchemaUtil.fromType(AuditProfileConfig.class),
                        SchemaUtil.fromType(AuditLogSplitterSettingsResource.class),
                        Set.of()));
    }

My use case is an audit splitter, retentionDays are defined in each audit profil pointed by the splitter. But it is not used for the splitter profil which is and audit profil too

image
image1037×684 27.1 KB

2

I’m seeing about getting someone more knowledgeable about field manipulation to weigh-in here, but we do have an open ticket to improve this very behavior. We had previously identified that the current “profile” settings for extension points are far too rigid (especially for Identity Providers), and will be providing something like a defaultProfile field in later versions which allow for overwriting the existing profiles.

3

I'm not sure the linked ticket is relevant here.

@mazeyrat this is the same as in 8.1, isn't it?

4

in 8.1, there was no default field for audit profile

5

@Kevin.Herron The linked ticket would allow for the user to define the profile of their extension point, which would in turn drive the UI.

6

AFAICT 14156 is what does that work, not 14157, and it allows them to provide a custom default value for the profile config, not redefine what fields the profile config has or what it looks like. The profile config belongs to the extension point type. This would allow him to set a different default retention period, not get rid of it entirely.