Bug 4723 AAA StoreBuilder init times out too early

AAA StoreBuilder will timeout if the backing data store isn't
loaded in 30s.  This is too short for some features.

Change-Id: I5c0c866b820bcf49e9592d620df69eefbb8dc2db
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Inherit version from parent

Since the bundle is using the same version as the parent version we can
inherit it and clean up the pom a little. Some aaa artifacts are already
using this method so this patch just makes the remaining ones update for
consistency.

Change-Id: I7af809d24475f096fbcedacbed1f239ff76fecec
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Add the new config artifact to feature dependencies.

AAA distribution job (and several downstream merge jobs)
are failing because they are unable to locate
org.opendaylight.aaa:aaa-h2-store:xml:config:0.3.0-SNAPSHOT
This is an attempt to fix that.

Also, the new config artifact is added to artifacts.

Change-Id: Id37064d6eb3eaf87778c0f7f3cec7cd9dcb463a3
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Merge "Revert "Revert "BUG 4624: Use default domain when credentials dont provide one"""

Revert "Revert "BUG 4624: Use default domain when credentials dont provide one""

This reverts commit 7bc9064d9fca906e397afadef2ab86838402ceb0.

Change-Id: I3a0ac71bb37540a16a037d43c6b150266bc1c6a7
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

H2 Store converted to Config Subsystem

Change-Id: I6400587b920f8558512b942889216f3c72e42a49
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Hardwire Services in sts ServiceLocator

Change-Id: Ia408c78484ee0183626437318c147d60979d204f
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Apply comments for MDSAL Store

This reverts commit 7050d9954aed918d6b6bca4ccaec803a33f83a1d.

Change-Id: I450370f72888a54790d2c3e1077adab96a22bc40
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Re-apply MD-SAL Data Store For AAA

This reverts commit 56837b89838429e326b8f18312f66331fe85eb44.

Change-Id: Ic8ec33213fbd8a632ab4b37a25fb1266f2f35d4f
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Merge "Add OAuth2 Token to Shiro to maintain cross-version compatibility"

Add OAuth2 Token to Shiro to maintain cross-version compatibility

Adds a mechanism to authenticate OAuth2 tokens, similar to
TokenAuthFilter's implementation.

Change-Id: Idade3da6fc364e1635d8a92b37e617d5ca697821
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Revert "MD-SAL Store for IDM"

This reverts commit 4a4dcdb561819b4016b918516d3dddf116016d0b.

Change-Id: I27df1cad7fa1c5e67f89cc84ca403e96fb5999e9
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Revert "BUG 4624: Use default domain when credentials dont provide one""

Merge "Revert "MD-SAL Store applied Comments""

Revert "BUG 4624: Use default domain when credentials dont provide one"

This reverts commit 4e67c246552d8bb40d5090bc0f572df91cce84ac.

Change-Id: I389426f9f95e6d6c91bc2b300dfe257af992f571
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Revert "MD-SAL Store applied Comments"

This reverts commit f429ddf8d4db71c40ca9957c52e4052b95cb10de.

Change-Id: Ifba2d2342cbb793d5bd20dff22b2a7c83629dcf9
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

BUG 4624: Use default domain when credentials dont provide one

Change-Id: I28a76301b2a588dd52d686d76f7abcace036ee88
Signed-off-by: Tomas Cere <tcere@cisco.com>

Merge "Enables TokenAuthRealm adapter to existing data stores"

Merge "MD-SAL Store applied Comments"

MD-SAL Store applied Comments

Change-Id: Idc25ccc288767ee7d97681734a5a16b78689729f
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Enables TokenAuthRealm adapter to existing data stores

Enables a new realm to check authentication against existing data
stores (IdM & MD-SAL).

Change-Id: I65bd61fe77afa09dd1c02ac37831c516b1cc8200
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Switch DefaultRBACRules to utilize lazy initialization

Change-Id: I6e8ecca904e2c64f6d977204f36b0f4c83275702
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Bug 4588 aaa-authn-api causing stack overflow for sfc tests

Swaps out the logging library in favor of the l2switch library, so
that the classpath is not polluted with APIs that do not behave
well together.

Change-Id: I067cfa648904a339b572fc3cd77e326bfcc1a209
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "MD-SAL Store for IDM"

MD-SAL Store for IDM

Change-Id: Ib44bf394616b5a471025a1d5b98ea3574f2e94f3
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Fix aaa-shiro JDK8 compatibility

Fixes how external links are documented.

Change-Id: I493476675aa7b4f36ca82d33c0469da5b1605fc7
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove duplicate dependency

Maven warns about multiple entries in dependecies -- remove them to fix
the warning.

Change-Id: Ic1cfa942dd44fee9a17cf09020a5cd98b0f0547f
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Fix JDK8 compatibility"

Fix JDK8 compatibility

I355acae1ac6652eb15cbd83ce9b0fcdcb9cc8afc caused compilation regression
in CredentialAuth. Fix that.

Change-Id: Ic523cf60dbafc681962d19c7cdb67b17cef69f89
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Revert "Bug 2515 - Jersey to support JAX-RS 2.0""

Revert "Bug 2515 - Jersey to support JAX-RS 2.0"

This reverts commit b1c7e0948abe0d62f130667ee98fd89b148a1009.

Change-Id: I4f1be95f9cab2f25a8309c305f8991a0faaccea2
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Bug 2515 - Jersey to support JAX-RS 2.0"

Remove unnecessary synchronization locks in service locator

ServiceLocator instance variables are already volatile to ensure
consistency across threads.  Thus, synchronization locks aren't
needed, since reads and writes are already atomic.

Change-Id: I3eb3bdb4944ecd4ca8835b86ca35f67682130e4e
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "DataEncrypter Unit Test and remove previous changes"

Add documentation to ServiceLocator classes regarding Activators

Documents some weird reflective behavior that previous authors included.
Some methods in the ServiceLocator classes are called by Activators
using reflection.

Change-Id: I374ee5887f5800e2fed26041108108f066db47ea
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Federation ServiceLocator to use accessor methods instead of public fields

Accessor methods shoould be preferred to public fields, as stated in
Effective Java 2 Item 14 (page 71).  This changes the aaa-authn-federation
bundle so that accessor methods are used instead of public instance variables.
Since accessor and setter methods are synchronized, this change also provides
thread safety, which is currently missing.

Change-Id: Ia048b8dcae1691c256b5e1404beb7cf4ce10ed62
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Adds Shiro Web-based authorization capabilities

Shiro supports RBAC for url patterns that may contain asterisks.  Some DOM
leaves should be filtered for security reasons, such as those containing AAA
data.  This allows installation of a set of default, immutable filters;  the
alternative was to allow installation in shiro.ini, but that file is mutable.

Since aaa-shiro isn't presently active, no changes will occur for external
projects.  This is setting up for when the aaa implementation is swapped out
for an Apache Shiro based one.

Change-Id: If0d037cafb90d8cf51713e538f41967f85f325ae
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

DataEncrypter Unit Test and remove previous changes

Change-Id: Ib8ef8d5e2532f96614cea2bd0efb663c69347953
Signed-off-by: melserngawy <melserngawy@inocybe.com>

Merge "unit test for authn-idp-mapping"

unit test for authn-idp-mapping

Change-Id: Ic720195734e18f8ddbea6e54713f70be2920b30f
Signed-off-by: melserngawy <melserngawy@inocybe.com>

Merge "unit test for AuthNStoreUtil, add check for null values to AuthNStoreUtil class methods and update POM file for test dependencies"

unit test for AuthNStoreUtil, add check for null values to AuthNStoreUtil class methods and update POM file for test dependencies

Change-Id: I0f2eec5c3500105854547d3216755c47a1b30cf7
Signed-off-by: melserngawy <melserngawy@inocybe.com>

Bug 2515 - Jersey to support JAX-RS 2.0

Upgrade versions of Jersey and JAX-RS.  Update tests appropriately.

Change-Id: Ia5aa836256f87bc268a8430944ab2f416965c9e0
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Use odlparent-lite as parent for artifact pom"

Merge "Bug4430 Unclear error message when Basic Authentication has a bad header format"

Bug4430 Unclear error message when Basic Authentication has a bad header format

This change ensures an AuthenticationException is thrown when and improperly
formatted Authentication Header is provided.  Tests are provided to ensure this
functionality.  The HttpBasicAuth.validate() function is refactored to utilize a
series of helper methods.

Change-Id: I2438651092c4bc018270ead0ccddd98ec10a3422
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

ServiceLocator to use accessor methods instead of public fields

Accessor methods shoould be preferred to public fields, as stated in
Effective Java 2 Item 14 (page 71).  This changes the aaa-authn-sts bundle so
that accessor methods are used instead of public instance variables.   Since
accessor and setter methods are synchronized, this change also provides
thread safety, which is currently missing.

Change-Id: I24bae3cfdb9525279539536876796bbd41ebf9af
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Use odlparent-lite as parent for artifact pom

Change-Id: Iabba38dcec50367e715a50a2ca0d8884d7955f40
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Merge "Support enabling and disabling shiro based AAA"

Use default sdn domain when credential domain is null

Change-Id: I4a3e0179475466d2ac2795564f71176189356c20
Signed-off-by: Tomas Cere <tcere@cisco.com>

Resubmit AAA Model Changes

Change-Id: I355acae1ac6652eb15cbd83ce9b0fcdcb9cc8afc
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Support enabling and disabling shiro based AAA

This change supports enabling and disabling shiro AAA through the use of a
ServiceProxy.  This support was added in order to support workflows such as
those included in the odl-restconf-noauth feature.  This change also adds a
significant amount of documentation to help users extend and debug the shiro
functionality.

Change-Id: I213a200c18d48fd1bf59da1ec171b0814a283fce
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Adds some documentation and sample files surrounding federation"

Merge "Bug 4313: Remove dependency on netconf"

Bug 4313: Remove dependency on netconf

aaa's features-aaa-authz still depends on netconf, which introduces a
cyclic dependency. Dropping the odl-restconf feature from
odl-aaa-authz fixes this.

Change-Id: I77a1de4ef2d9d8892c5fd3886966c8a309dbf953
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Adds some documentation and sample files surrounding federation

Some documentation and a few sample files to set up a simple
federated AAA scenario.

Change-Id: I9442e4ada2c718c88f9ca576f0959880f0043372
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Revert "Revert "Formatting fixes""

This reverts commit db480102f6a9496ada7cfa7354ed88f4ea313d24

Change-Id: Id04fe9e09ecac8d2506669cc7b83449048272b24
Signed-off-by: Wojciech Dec <wdec@cisco.com>

Merge "Introduction of the odl-aaa-shiro feature"

Revert "AAA Model Changes"

Revert until feature loading is addressed.

This reverts commit a5c6bbab17ffe1cc712f9c7566efbed1a258250e.

Change-Id: I7415a6a1df6382415db404229f41c292e246136d
Signed-off-by: Wojciech Dec <wdec@cisco.com>

Merge "Bug 4289 AuthZ config subsystem files are loaded to the wrong place"

Merge "AAA Model Changes"

Bug 4289 AuthZ config subsystem files are loaded to the wrong place

AuthZ config subsystem files are loaded to ${KARAF_HOME} instead of
${KARAF_HOME}/etc/opendaylight/karaf.

Change-Id: Ib0c9b440d40930745913be4af1923dd50369aece
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Migration to use MD-SAL Project"

AAA Model Changes

Change-Id: I76f8d2405a57480d484a7dc8df5e44c71de40314
Signed-off-by: Wojciech Dec <wdec@cisco.com>

Introduction of the odl-aaa-shiro feature

Introduces the odl-aaa-shiro feature, based on the aaa-shiro bundle.  This
bundle is built separately from the rest of the AAA subsystem, and can stand
independent of other code in AAA.  Without appropriate changes to restconf
in the netconf project, this change is not usable in the ODL project.

This sets the AAA scaffolding for aaa-shiro integration.  This patch can
be merged without affecting other code in the aaa or restconf projects.  It
is the start to migrating from the custom AAA solution to one that is built
on Apache Shiro.  This patch supports 1.X versions of JAX-RS at this time,
and will later provide functionality to support the 2.X API.

This introduction of code adds/allows the following Realms:
1) ODLJndiLdapRealm
2) INI file based
3) JDBCRealm

There is stub support for:
1) RadiusRealm
2) TACACSRealm
But these have not yet been implemented.  They stand as a Proof of Concept
as to how easy it is to create different AAA backends to authenticate against.

Corresponding test methods are present for all testable methods in classes.
This is the first of many incremental patches to help migrate AAA to Shiro.
This particular patch will not actually affect the current implementation.

Change-Id: Ifbdec290d87434279882295c24e35412b641154f
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Migration to use MD-SAL Project

Change-Id: I8490822ef10962d5bbd48fe4fd27092a4c26b916
Signed-off-by: Tony Tkacik <ttkacik@cisco.com>

Revert "Formatting fixes"

This reverts commit 7803df99b6b61c43048ccbc20afb6f640a53c0a3.

Change-Id: Ic91845013379c710238070c88526302a8a3bdc82
Signed-off-by: Wojciech Dec <wdec@cisco.com>

Merge "Remove pluginManagement and clean up variables"

Merge "Formatting fixes"

Remove pluginManagement and clean up variables

The parent POM's pluginManagement is redundant with odlparent's, so
remove it. (I'm working on reducing the number of variables which need
to be exported from odlparent.)

A few other variables aren't used, remove them too.

Change-Id: I4c9e41633527457054661f35edb4e1077108d6d7
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Formatting fixes

Change-Id: I213628f223267a0bb185ebccdb7230c71658d6ef
Signed-off-by: Wojciech Dec <wdec@cisco.com>

Fix  to checkstyle configuration

Change-Id: I0ad8aa77b362b717d968f0b872b1c0e1227e3f7a
Signed-off-by: Wojciech Dec <wdec@cisco.com>

Resolve cyclic dependency with netconf

Move aaa-authn-odl-plugin to netconf to prevent cyclic dependency.
Migrate restconf dependencies to new groupId

Change-Id: I7931dbee55199ea4cc0b11f4f32e78da9eb4c9c1
Signed-off-by: Tomas Cere <tcere@cisco.com>

Bug 3680 - Adding post activation service registration callbacks

Change-Id: Ia413410ae139d6b0fec1bb40aba5d4d345158561
Signed-off-by: Wojciech Dec <wdec@cisco.com>

Merge "Bug4168 Deprecate inappropriate uses of StringBuilder"

Merge "Bug 4146 odl-aaa-authz does not work for Write or ReadWrite Transactions"

Bug 4146 odl-aaa-authz does not work for Write or ReadWrite Transactions

This patch modifies the authorization model by defining ActionTypes that are
applicable to the existing DOMDataReadOnlyTransaction,
DOMDataWriteOnlyTransaction and DOMDataReadWriteTransaction.  The ActionType
enum is utilized to add authorization to the AuthzWriteOnlyTransaction and
AuthzReadWriteTransaction.  Utility methods for ensuring authorization are
added to the AuthzServiceImpl.

Change-Id: Ia5c67946160e3dc0a074e616fbbb980e5eeafec1
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Bug4168 Deprecate inappropriate uses of StringBuilder

Patches old code to replace StringBuilder invocations with String
concatenation for performance reasons.

Change-Id: I048c73969c6251b77a2c90a0fdd8825f59dd01dd
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Merge "Adds a folder for postman collections and a basic authz example"

Merge "Add config subsystem binding to aaa-authn-odl-plugin"

Adds a folder for postman collections and a basic authz example

Adds authz postman collection for easy import/use.  These rest
endpoints are just scaffolding right now;  they do not actually
affect the Java code

Change-Id: I2e4c59c0eceb334ac5758cd8507027af81842c45
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Fix license header violations in aaa-idmlight

Change-Id: Ib5148d1690134501cd14a47b9b991aabe5ee30bc
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn

Change-Id: Id888328a85d146249e87b1a074a64fbbf6a0464a
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-basic

Change-Id: Id2a8f0fbfb53cc6c4d4d37b103da18f757c63063
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-keystone

Change-Id: Ic4b63252428daf95d28f756f4d7800afd5b8ffe2
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-odl-plugin

Change-Id: Ia0b7cc1cd97318f370a2433b2c8ce6c912d8540e
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-sssd

Change-Id: I9f8329410f5a809236085d6ad178b772f8319ae7
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-federation

Change-Id: Ic1443b2c38581fa3fecf419899c80693187ff3e7
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-store

Change-Id: I60b24c366231dcd8f0b94e21fbe54c804a68b4fb
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-sts

Change-Id: I744f224f9df3beaa82459fbdafce342fab0935bc
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-idp-mapping

Change-Id: Ifa23217964ae86d192ab271837474b054b86e9eb
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn

Change-Id: Ic55740e6ec763686a8a377cc4a7eeaffbf8a6bac
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Fix license header violations in aaa-authn-api

Change-Id: If99adb015c3deebd5615a0d3ca267447428fbe8b
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Add config subsystem binding to aaa-authn-odl-plugin

AuthProvider is now a regular module instantiated automatically as a default
instance.

Change-Id: I3244242bfe3d1be1207ab077d4e5cc107a33ee4e
Signed-off-by: Maros Marsalek <mmarsale@cisco.com>

Merge "Bug 4054 Federation RuleProcessor utilizes System.out which interferes with karaf"

Merge "Fix Bug 3924 - aaa passwords are stored in clear text"

Bug 4054 Federation RuleProcessor utilizes System.out which interferes with karaf

Changes System.out calls to logger.info().

Change-Id: I20ef61d59e2a310de8705e165b881659063d6b32
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Fix Bug 3924 - aaa passwords are stored in clear text

Change-Id: If5d370166011b85f82b54b21c273028b90b30ea1
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Merge "Explicitly set git-review branch to master"

Merge "Bug 4038 Grant Creation is Broken"