karaf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Freeman Fang (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (KARAF-4917) Support dynamicaliy registered configurations in RBAC
Date Wed, 04 Jan 2017 02:26:58 GMT

    [ https://issues.apache.org/jira/browse/KARAF-4917?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15796930#comment-15796930
] 

Freeman Fang commented on KARAF-4917:
-------------------------------------

Hi Zoran,

First of all, thanks for the contribution.

Could you please elaborate why you need use ConfigurationAdmin#createFactoryConfiguration
to configure the RBAC. IIRC the ConfigurationAdmin#createFactoryConfiguration is used for
the multiple configuration(that's why the random UUID comes from) but the RBAC not need this
IMO, why not just use ConfigurationAdmin#getConfiguration instead. That said, the RBAC configuration
should be PID but not the factoryPid.

Freeman

> Support dynamicaliy registered configurations in RBAC
> -----------------------------------------------------
>
>                 Key: KARAF-4917
>                 URL: https://issues.apache.org/jira/browse/KARAF-4917
>             Project: Karaf
>          Issue Type: Improvement
>          Components: karaf-security
>    Affects Versions: 4.1.0
>            Reporter: Zoran Regvart
>            Assignee: Freeman Fang
>            Priority: Minor
>
> When adding configuration at runtime through {{ConfigurationAdmin#createFactoryConfiguration}}
Apache Felix appends [random|https://github.com/apache/felix/blob/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/ConfigurationManager.java#L397-L400]
[UUID|https://github.com/apache/felix/blob/trunk/configadmin/src/main/java/org/apache/felix/cm/impl/ConfigurationManager.java#L1093-L1137]
to the Configuration PID.
> In order for those to be considered by {{getRequiredRoles}} that random UUID needs to
be removed in comparison performed by {{getGeneralPid}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message