You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Define common beans as part of an auto-configuration to reduce manual configurations on clients side.
Actual behaviour
Clients should define objects/beans in their code to use the library, such as OPAClient and OPAAuthorizationManager.
It would be nice to have a more streamlined experience by defining pre-configured beans in an @AutoConfiguration class
which will be created in client applications by adding that can be used by clients.
How to improve
Define OPAAutoConfiguration class that will create the required beans:
Moreover, considering @ConditionalOnMissingBean annotations, clients can define their own OPAClient and OPAAuthorizationManager beans. Fewer clients would be affected by changing the library's internal structure (e.g. constructors). Furthermore, OPAProperties can be used to reference externalized properties (depends on #21).
The text was updated successfully, but these errors were encountered:
Expected behaviour
Define common beans as part of an auto-configuration to reduce manual configurations on clients side.
Actual behaviour
Clients should define objects/beans in their code to use the library, such as
OPAClient
andOPAAuthorizationManager
.It would be nice to have a more streamlined experience by defining pre-configured beans in an
@AutoConfiguration
classwhich will be created in client applications by adding that can be used by clients.
How to improve
Define
OPAAutoConfiguration
class that will create the required beans:Then, clients can use the library without defining
OPAClient
andOPAAuthorizationManager
beans/objects:Moreover, considering
@ConditionalOnMissingBean
annotations, clients can define their ownOPAClient
andOPAAuthorizationManager
beans. Fewer clients would be affected by changing the library's internal structure (e.g. constructors). Furthermore,OPAProperties
can be used to reference externalized properties (depends on #21).The text was updated successfully, but these errors were encountered: