aaa.git
6 years agoRemove shiro.ini conversion script 49/66949/1
Ryan Goulding [Mon, 8 Jan 2018 17:18:17 +0000 (12:18 -0500)]
Remove shiro.ini conversion script

shiro.ini was done away with many releases ago.  As part
of its removal, the AAA team added a conversion script to
convert the shiro.ini file into a format that could be
recognized by the aaa clustered-app-config.  Since the
shiro.ini based approach has been gone for over a release,
this patch removes the conversion script since it should
no longer be needed.

Change-Id: I6866ae1faea0362251ff26fabc8f6df360acde08
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMoon Authorization Driver for ODL/AAA 15/66615/6
YuchenWang [Tue, 19 Dec 2017 14:05:45 +0000 (22:05 +0800)]
Moon Authorization Driver for ODL/AAA

https://github.com/WangYuchenSJTU/ODL-Moon-Authz

Change-Id: I7cc569561c7dd3cee26985b11c27ba9dbf913642
Signed-off-by: YuchenWang <lucassjtu@gmail.com>
6 years agoMerge "AAA-154 Make H2 database credentials configurable"
Ryan Goulding [Fri, 1 Dec 2017 19:44:10 +0000 (19:44 +0000)]
Merge "AAA-154 Make H2 database credentials configurable"

6 years agoCollapse features into features-aaa repository 16/66016/3
Ryan Goulding [Tue, 28 Nov 2017 17:59:59 +0000 (12:59 -0500)]
Collapse features into features-aaa repository

Instead of providing two of essentially the same repository,
instead just provide one called features-aaa.

Change-Id: Ic60fca3a2b3129fe424e353ea1a1e074927f0a98
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "Clear claim cache when grants/roles/domains change"
Ryan Goulding [Wed, 29 Nov 2017 14:02:31 +0000 (14:02 +0000)]
Merge "Clear claim cache when grants/roles/domains change"

6 years agoMerge "Use gson to parse JSON output"
Ryan Goulding [Wed, 29 Nov 2017 13:57:21 +0000 (13:57 +0000)]
Merge "Use gson to parse JSON output"

6 years agoUse gson to parse JSON output 55/66055/1
Robert Varga [Wed, 29 Nov 2017 11:01:33 +0000 (12:01 +0100)]
Use gson to parse JSON output

org.json is poorly maintained and licensed piece of software,
which is causing us headaches whenever encountered. Remove its use
and use GSON to parse JSON payloads.

Change-Id: I07a7cc486b88e949b23b1d8714163eabea116b92
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
6 years agoClear claim cache when grants/roles/domains change 36/66036/1
Ryan Goulding [Tue, 28 Nov 2017 22:00:19 +0000 (17:00 -0500)]
Clear claim cache when grants/roles/domains change

The claim cache should be cleared whenever a change
occurs to these entitites.

Change-Id: Iadfc71e219847609147b64394fd3b549c7979342
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoAAA-154 Make H2 database credentials configurable 33/66033/1
Ryan Goulding [Tue, 28 Nov 2017 21:33:04 +0000 (16:33 -0500)]
AAA-154 Make H2 database credentials configurable

Makes the credentials for H2 file-oriented database
configurable.  These aren't AAA credentials, but rather
the credentials to the underlying data store.

Change-Id: I5e511a172155e355e3f88f2145f7ef95b4896eb7
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "Fix stale javadoc in StoreBuilder"
Ryan Goulding [Tue, 28 Nov 2017 19:41:31 +0000 (19:41 +0000)]
Merge "Fix stale javadoc in StoreBuilder"

6 years agoFix stale javadoc in StoreBuilder 20/66020/1
Ryan Goulding [Tue, 28 Nov 2017 18:51:34 +0000 (13:51 -0500)]
Fix stale javadoc in StoreBuilder

Probably a rebase issue, but the javadoc has two @return
annotations.  This deletes the stale/irrelevant one.

Change-Id: I302dbc5b601ad52897b03c3b15583b0ebe8caf6f
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoAAA-153 remove "user" OOB account 17/66017/1
Ryan Goulding [Tue, 28 Nov 2017 18:40:58 +0000 (13:40 -0500)]
AAA-153 remove "user" OOB account

Limit exposure by just providing "admin" account OOB.

Change-Id: I451d84e06a208e5b9701fe461b5b46ef47fcbcdd
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "AAA-151: Invalidate claim cache for CLI initiated changes"
Ryan Goulding [Tue, 28 Nov 2017 17:20:31 +0000 (17:20 +0000)]
Merge "AAA-151:  Invalidate claim cache for CLI initiated changes"

6 years agoMerge "MdsalRealm enable/disable user"
Ryan Goulding [Tue, 28 Nov 2017 16:38:19 +0000 (16:38 +0000)]
Merge "MdsalRealm enable/disable user"

6 years agoAAA-151: Invalidate claim cache for CLI initiated changes 15/65815/5
Ryan Goulding [Tue, 21 Nov 2017 18:46:34 +0000 (13:46 -0500)]
AAA-151:  Invalidate claim cache for CLI initiated changes

The AAA IDM REST handlers already invoke the clearing of the
IdmLightProxy claimCache upon user update, but the CLI Handler did
not handle this properly.  This change manually invokes the claim
invalidation.

Access to AAA CLI commands requires access to the karaf console via
client or SSH.  Thus, system level access is needed to some degree
to invoke such commands.  The major threat involves the action of
an administrator changing a user password via CLI, and then still
being able to invoke the old password because the claimCache is not
purged.  This is why a reboot of karaf solves this issue;  the
claimCache is flushed.  We should not expect our users/administrators
to assume this in the future;  this change explicitly invokes
the claimCache flush.

AAA-151 affects Carbon, Nitrogen, and master.

Change-Id: Ie3101d4dc2bbb6d0db91ddad4d567a41d041150d
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMdsalRealm enable/disable user 82/65682/2
Ryan Goulding [Fri, 17 Nov 2017 19:38:21 +0000 (14:38 -0500)]
MdsalRealm enable/disable user

This was built into the original IdM data model for the H2 store,
and thus was cascaded to the Mdsal model for backwards compatibility
purposes mostly.  The original model didn't do anything with this
field;  this one now properly honors the "enabled" field.

Change-Id: I4f15c45a336008c66423ef438c141cf6df132583
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "Dissuade use of author name in source"
Ryan Goulding [Tue, 21 Nov 2017 15:29:05 +0000 (15:29 +0000)]
Merge "Dissuade use of author name in source"

6 years agoMerge "Add MdsalRealm information to README.md"
Ryan Goulding [Tue, 21 Nov 2017 14:50:30 +0000 (14:50 +0000)]
Merge "Add MdsalRealm information to README.md"

6 years agoDissuade use of author name in source 16/65116/5
Ryan Goulding [Fri, 3 Nov 2017 17:34:40 +0000 (13:34 -0400)]
Dissuade use of author name in source

This is just an attempt to make the code more community centric.
I don't care if other contributors want to keep their "@author"
annotation tags-- but I do want to spread the love in terms of
support queries etc :).

This removes my name/email anywhere possible in an effort to
make the aaa codebase more community centric.  I still plan to
be active, I just would rather people email the aaa reflector:

aaa-dev@lists.opendaylight.org

rather than unicast my email address.

Whenever I saw trivial changes like unused import or whatnot,
I just removed them to make the code cleaner.

Change-Id: I56573bf618deac22c41a2276233ed9d6c00c07de
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoFix odlparent-3.0.0 checkstyle violations 54/65754/1
Robert Varga [Mon, 20 Nov 2017 18:30:34 +0000 (19:30 +0100)]
Fix odlparent-3.0.0 checkstyle violations

There are new violations reported:
- hidden fields
- utility classes with public constructor

Change-Id: Iced411566cc5e1d8c5d69554222db0d53bedee3f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
6 years agoRemove duplicate dependency declarations 53/65753/1
Robert Varga [Mon, 20 Nov 2017 18:34:39 +0000 (19:34 +0100)]
Remove duplicate dependency declarations

This fixes maven warnings related to duplicates being declared.

Change-Id: I9b2c905b893de73466bd7f2012b1e53fb0ba236f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
6 years agoAdd MdsalRealm information to README.md 81/65681/1
Ryan Goulding [Fri, 17 Nov 2017 19:27:34 +0000 (14:27 -0500)]
Add MdsalRealm information to README.md

Add caveats about the new MdsalRealm to the AAA README.  Point
out the current state and the future work involved to get
MdsalRealm in production shape.

Change-Id: I55016467e37f00c14d4ade2b8c113032c6c9a6a8
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agomdsalrealm based on shiro 08/65508/7
Ryan Goulding [Tue, 14 Nov 2017 16:49:17 +0000 (11:49 -0500)]
mdsalrealm based on shiro

Adds basic support for MdsalRealm, a realm which authenticates
using information from the "authentication" container in the
"aaa.yang" model.  MdsalRealm still isn't enabled OOB.

aaa endpoints are protected by allowing access to admin role only.

Change-Id: I3a6423e28c0e14764d172698aa4d67d0ec472379
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "AAA-149: migrate ServiceProxy to the correct package name"
Mohamed El-Serngawy [Fri, 17 Nov 2017 14:15:56 +0000 (14:15 +0000)]
Merge "AAA-149: migrate ServiceProxy to the correct package name"

6 years agoMerge "Rid AAA source of binary images"
Mohamed El-Serngawy [Fri, 17 Nov 2017 14:15:28 +0000 (14:15 +0000)]
Merge "Rid AAA source of binary images"

6 years agoCollapse aaa feature repositories 35/65635/1
Ryan Goulding [Thu, 16 Nov 2017 19:46:22 +0000 (14:46 -0500)]
Collapse aaa feature repositories

AAA advertises two feature repositories.  Historically, this was done based
on the fact that it was hoped to pull out a ton of the stuff surrounding
non-shiro implementation in downstream distributions by just discluding the
non-shiro feature repository.  However, it appears that this has limited use,
and primarily serves as a distraction.  Furthermore, there are more
intelligent ways to perform such an operation.

The first part of this change renames the "authn" module to "aaa", because
the contained features repository is "features-aaa" and is not specific
to authentication.

The second part moves odl-aaa-shiro to the aaa module.  This is done in
preparation to remove features-aaa-shiro, which will no longer be offered.

features-aaa-shiro still contains the odl-aaa-shiro feature for now.
After all downstreams are migrated to utilize features-aaa instead, it
will be completely removed as a module.  This is a multistep process and
should not affect downstream consumers.

Change-Id: Ie084b622b1e58c661fee910e85a204436bf23f3e
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoAAA-149: migrate ServiceProxy to the correct package name 21/65621/2
Ryan Goulding [Thu, 16 Nov 2017 14:26:59 +0000 (09:26 -0500)]
AAA-149: migrate ServiceProxy to the correct package name

This migrates ServiceProxy to the correct package.  AAA-149 is
being done in several small steps in order to ensure that the
changes do not affect downstream consumers.  This change was
tested against NETCONF to ensure there was no downstream
breakages (which makes sense since NETCONF does not directly
manipulate ServiceProxy and actually only affects it through
AAA code).

Change-Id: If142f4fe1ddc91cf844fb8d0ca3a6496f0828efd
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRid AAA source of binary images 28/65628/1
Ryan Goulding [Thu, 16 Nov 2017 18:22:39 +0000 (13:22 -0500)]
Rid AAA source of binary images

From long ago we referenced some images in our README.md and directly
stored them in source.  Thats a bad idea for a number of reasons.  This
simply gets rid of the references to the one remaining image and the
source/output graphics that were stored in source.  If we want them back,
we can figure out a way to upload them to somewhere more appropriate.

The one existing accurate diagram, direct_authn.png, was really simple
to understand anyway.  If we want, we can restore it somewhere new,
put it in ascii-art, or just describe it better.  Also, it is probably
better to consolidate that sort of information in the docs project.

Change-Id: I104c32eb844cf4031f26f6108aeb8b98d13b9401
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "AAA-149 migrate Handler classes to correct package"
Mohamed El-Serngawy [Thu, 16 Nov 2017 15:30:25 +0000 (15:30 +0000)]
Merge "AAA-149 migrate Handler classes to correct package"

6 years agoMerge "Remove unused graphics"
Mohamed El-Serngawy [Thu, 16 Nov 2017 15:26:59 +0000 (15:26 +0000)]
Merge "Remove unused graphics"

6 years agoMerge "Fix compile time warnings in aaa-shiro-act"
Mohamed El-Serngawy [Thu, 16 Nov 2017 15:26:20 +0000 (15:26 +0000)]
Merge "Fix compile time warnings in aaa-shiro-act"

6 years agoAAA-149 migrate Handler classes to correct package 15/65615/1
Ryan Goulding [Thu, 16 Nov 2017 13:32:34 +0000 (08:32 -0500)]
AAA-149 migrate Handler classes to correct package

Migrate the *Handler classes to the right package, along with
the corresponding tests.

Change-Id: I1e01f4bfec298fb81376a8ea9f8bc8927c44f4b1
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoFix compile time warnings in aaa-shiro-act 55/65555/1
Ryan Goulding [Wed, 15 Nov 2017 15:56:49 +0000 (10:56 -0500)]
Fix compile time warnings in aaa-shiro-act

The init/close methods in the Provider for aaa-shiro-act are called via
blueprint, but IntelliJ and other IDEs do not recognize our custom blueprint
path (src/main/resources/org/opendaylight/blueprint).  Thus, we get a ton
of annoying warnings about unused init/close methods.  This fixes the issue
for aaa-shiro-act.

Change-Id: Iefd1aa44bf85c9e073d080c9293c325da82b3c33
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoFix weird formatting in blueprint.xml file 54/65554/1
Ryan Goulding [Wed, 15 Nov 2017 15:52:46 +0000 (10:52 -0500)]
Fix weird formatting in blueprint.xml file

aaa-shiro-act had an empty bean definition so I fixed the notation to use
self-enclosed xml.  This is just cleaner and was really bothering me :).

Change-Id: I8a08ffbb338f935e6db0800a05bc8867b789f090
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRemove unused graphics 53/65553/1
Ryan Goulding [Wed, 15 Nov 2017 15:47:21 +0000 (10:47 -0500)]
Remove unused graphics

Previously referenced in the README, but now are no longer applicable.
These graphics depicted sssd plugin which was removed many releases ago.
Furthermore, graphics really don't belong in source (i.e., git).

Change-Id: If78bc55f6ee6b42e6abb28df356baa9aeda122f6
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRefactor AAA datastore & add datastore change functionality 63/63863/9
Mohamed ElSerngawy [Sun, 1 Oct 2017 20:17:37 +0000 (16:17 -0400)]
Refactor AAA datastore & add datastore change functionality

Moving h2 store bundle to aaa-shiro bundle and
add the option to switch between data stores

Change-Id: I4a3ce831eecae4c1382852fc004149da3abe6d57
Signed-off-by: Mohamed ElSerngawy <m.elserngawy@gmail.com>
6 years agoRemove stale documentation 30/65230/1
Ryan Goulding [Mon, 6 Nov 2017 20:33:25 +0000 (15:33 -0500)]
Remove stale documentation

README.md contained some references to installing the filter as
a ContainerRequestFilter.  Now, we actually use a regular
javax.servlet.Filter, so this documentation is out of date.
This patch just removes the stale documentation.

Change-Id: I7a5183f7db0c77e2012c03e754e7428ebab69098
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "Revert "Remove felix.dm usage in aaa-filterchain""
Ryan Goulding [Mon, 6 Nov 2017 20:17:03 +0000 (20:17 +0000)]
Merge "Revert "Remove felix.dm usage in aaa-filterchain""

6 years agoMerge "Revert "Remove felix.dependencymanager from Shiro POM""
Ryan Goulding [Mon, 6 Nov 2017 20:16:53 +0000 (20:16 +0000)]
Merge "Revert "Remove felix.dependencymanager from Shiro POM""

6 years agoMerge "Revert "Remove un-used felix.dm usage from aaa-encrypt-service""
Ryan Goulding [Mon, 6 Nov 2017 20:16:43 +0000 (20:16 +0000)]
Merge "Revert "Remove un-used felix.dm usage from aaa-encrypt-service""

6 years agoMerge "Revert "Remove felix.dm from odl-aaa-shiro feature""
Ryan Goulding [Mon, 6 Nov 2017 20:16:32 +0000 (20:16 +0000)]
Merge "Revert "Remove felix.dm from odl-aaa-shiro feature""

6 years agoRevert "Remove felix.dm usage in aaa-filterchain" 29/65229/1
Ryan Goulding [Mon, 6 Nov 2017 20:16:11 +0000 (20:16 +0000)]
Revert "Remove felix.dm usage in aaa-filterchain"

This reverts commit 822554dcc7cfd59034622a0f41df5b25a2ae2520.

Change-Id: I0ea057fa13784849849020906c81add04cebad01
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRevert "Remove felix.dependencymanager from Shiro POM" 28/65228/1
Ryan Goulding [Mon, 6 Nov 2017 20:16:01 +0000 (20:16 +0000)]
Revert "Remove felix.dependencymanager from Shiro POM"

This reverts commit 8efc8bc00d2cfbd0d983bf36eec578cfddc60c9e.

Change-Id: I1ff4fef8f5c2fb3d1cf5d1e64a944c5c0f5a2f20
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRevert "Remove un-used felix.dm usage from aaa-encrypt-service" 27/65227/1
Ryan Goulding [Mon, 6 Nov 2017 20:15:47 +0000 (20:15 +0000)]
Revert "Remove un-used felix.dm usage from aaa-encrypt-service"

This reverts commit 38887453198b93327e7d298ad33dd697c4070b28.

Change-Id: Ic419b0e6f1913da2078799002e9df46b38e67b55
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRevert "Remove felix.dm from odl-aaa-shiro feature" 26/65226/1
Ryan Goulding [Mon, 6 Nov 2017 20:09:42 +0000 (20:09 +0000)]
Revert "Remove felix.dm from odl-aaa-shiro feature"

This reverts commit fb462f2322b4156bb617ac041061612e50420020.

Change-Id: Id236b1ff1c8083e40700518b3255734c60ea4bab
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "Remove un-used felix.dm usage from aaa-encrypt-service"
Ryan Goulding [Mon, 6 Nov 2017 19:37:00 +0000 (19:37 +0000)]
Merge "Remove un-used felix.dm usage from aaa-encrypt-service"

6 years agoMerge "Remove felix.dm usage in aaa-filterchain"
Ryan Goulding [Mon, 6 Nov 2017 19:36:47 +0000 (19:36 +0000)]
Merge "Remove felix.dm usage in aaa-filterchain"

6 years agoMerge "Remove felix.dm from odl-aaa-shiro feature"
Ryan Goulding [Mon, 6 Nov 2017 19:36:40 +0000 (19:36 +0000)]
Merge "Remove felix.dm from odl-aaa-shiro feature"

6 years agoRemove felix.dm from odl-aaa-shiro feature 69/65169/1
Michael Vorburger [Mon, 6 Nov 2017 14:18:53 +0000 (15:18 +0100)]
Remove felix.dm from odl-aaa-shiro feature

Change-Id: Ie0024c763af881287770b06fe2e6bf67ed56afad
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
6 years agoRemove un-used felix.dm usage from aaa-encrypt-service 67/65167/1
Michael Vorburger [Mon, 6 Nov 2017 14:16:06 +0000 (15:16 +0100)]
Remove un-used felix.dm usage from aaa-encrypt-service

Change-Id: Ida4af1202ae0c46a07169e7fe0a10ce181c139c3
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
6 years agoRemove felix.dm usage in aaa-filterchain 65/65165/1
Michael Vorburger [Mon, 6 Nov 2017 13:44:54 +0000 (14:44 +0100)]
Remove felix.dm usage in aaa-filterchain

Change-Id: Ia25aa964992378fac014273db3f9596d2c28a693
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
6 years agoRemove felix.dependencymanager from Shiro POM 60/65160/1
Michael Vorburger [Mon, 6 Nov 2017 13:06:08 +0000 (14:06 +0100)]
Remove felix.dependencymanager from Shiro POM

and remove duplicate artifacts to remove Maven warnings

Change-Id: Ibe5cdb7a6d15365e8fe684d6b77f819f15fa8ba2
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
6 years agoMerge "remove odl-aaa-authn in favor of odl-aaa-shiro"
Ryan Goulding [Fri, 3 Nov 2017 20:23:20 +0000 (20:23 +0000)]
Merge "remove odl-aaa-authn in favor of odl-aaa-shiro"

6 years agoremove odl-aaa-authn in favor of odl-aaa-shiro 21/65121/2
Ryan Goulding [Fri, 3 Nov 2017 18:40:12 +0000 (14:40 -0400)]
remove odl-aaa-authn in favor of odl-aaa-shiro

Nothing uses odl-aaa-authn.  Remove it since it is just a wrapper
for odl-aaa-shiro since Nitrogen.

Change-Id: I143b036d79da92e02d3bd979a6284971275e7479
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoBug 8717 Remove the existing non-functional mdsal impl 19/65119/2
Ryan Goulding [Fri, 3 Nov 2017 18:05:46 +0000 (14:05 -0400)]
Bug 8717 Remove the existing non-functional mdsal impl

This implementation was designed by someone who has long since
left the project and has not been maintained since Lithium release
minus minor fixes to service activation and AAA API changes.

It was deprecated last release with:
https://git.opendaylight.org/gerrit/#/c/59189/

This removes the implementation so it cannot be used.  We
plan on adding a revamped version of the MDSAL store in
Oxygen timeframe.

Change-Id: I705924aac88ea317e1378fbeee4117f52d5e3904
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRemove unused concepts from aaa-authn-api 18/65118/1
Ryan Goulding [Fri, 3 Nov 2017 17:58:26 +0000 (13:58 -0400)]
Remove unused concepts from aaa-authn-api

There are several areas of the original aaa-authn-api
that were superfluous and never used.  This removes them
to minimize unused LOC, and hopefully improve SONAR scores.

Change-Id: I7eb3c31f8ee0d733f057f587319b2239c55c83ad
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoMerge "Cleanup existing README content"
Ryan Goulding [Fri, 3 Nov 2017 17:46:52 +0000 (17:46 +0000)]
Merge "Cleanup existing README content"

6 years agoMerge "Remove ClientService implementation"
Ryan Goulding [Fri, 3 Nov 2017 17:46:35 +0000 (17:46 +0000)]
Merge "Remove ClientService implementation"

6 years agoCleanup existing README content 14/65114/1
Ryan Goulding [Fri, 3 Nov 2017 17:13:19 +0000 (13:13 -0400)]
Cleanup existing README content

More content will be added later, this just fixes
what is there now.

Change-Id: Id9ee61aecc79685986ec26d1e6a38219e16de980
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRemove innacurate information in README 12/65112/1
Ryan Goulding [Fri, 3 Nov 2017 16:38:55 +0000 (12:38 -0400)]
Remove innacurate information in README

The README contained outdated information referencing
SSSD implementation, which was removed several releases
ago.  This removes corresponding advertisement to remain
truthful and accurate to our users.

Change-Id: I80845c738ca1291707b8b1fffd32f6765da676a0
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRemove ClientService implementation 11/65111/1
Ryan Goulding [Fri, 3 Nov 2017 16:34:43 +0000 (12:34 -0400)]
Remove ClientService implementation

ClientService was added a long time ago by contributors no
longer on the project, and has no known upstream users. Instead,
operators should create a service user using the idm endpoints.
This removes the default ClientService implementation since
it is unused.

Change-Id: Ic7fc5fc9aece532d7cdd754e9aa8f37f9ed1fc35
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agocleanup shiro features formatting 03/65103/1
Ryan Goulding [Fri, 3 Nov 2017 15:11:12 +0000 (11:11 -0400)]
cleanup shiro features formatting

Cleanup the shiro features prior to jersey2 migration.
The actual dependency logic will be resolved in the upgrade.

Change-Id: Iee9f23ae8def0d080ae8086749573c71a5400186
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoCleanup aaa-shiro/impl pom 01/65101/1
Ryan Goulding [Fri, 3 Nov 2017 15:07:08 +0000 (11:07 -0400)]
Cleanup aaa-shiro/impl pom

Prior to the jersey2 upgrade, clean up the pom from a
formatting perspective.  Dependencies will be cleaned
up during the migration to jersey2.

Change-Id: I5c7d024f7c815c8860af6eb116ad4b9d4232c03b
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoBUG-9261: add PKIUtil decrypt overloads for StringReader 57/64657/1
Tomas Cere [Tue, 24 Oct 2017 12:26:50 +0000 (14:26 +0200)]
BUG-9261: add PKIUtil decrypt overloads for StringReader

Change-Id: If22b57e300873211887deb9b0eb718b5482b764c
Signed-off-by: Tomas Cere <tcere@cisco.com>
6 years agoRemove unused pom properties 08/64208/1
Ryan Goulding [Thu, 12 Oct 2017 16:21:32 +0000 (12:21 -0400)]
Remove unused pom properties

Change-Id: Ia05f780948dcedbbc9958a16fdd567264dfb8028
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoBump odlparent 2.0.4 to 2.0.5 29/63629/2
Stephen Kitt [Wed, 27 Sep 2017 13:35:01 +0000 (15:35 +0200)]
Bump odlparent 2.0.4 to 2.0.5

Change-Id: Ice0bf02276a15a30fd98338f91591eb1f203aaa2
Signed-off-by: Stephen Kitt <skitt@redhat.com>
6 years agoBump aaa to use yangtools 1.2.0 41/63741/3
Anil Belur [Thu, 28 Sep 2017 03:11:41 +0000 (13:11 +1000)]
Bump aaa to use yangtools 1.2.0

Jira: releng-485
Change-Id: Icbb236dbce466a396ffd8960437116db3935a544
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
6 years agoMerge "Move idmlight bundle to aaa-shiro bundle"
Ryan Goulding [Wed, 27 Sep 2017 13:07:38 +0000 (13:07 +0000)]
Merge "Move idmlight bundle to aaa-shiro bundle"

6 years agoMove idmlight bundle to aaa-shiro bundle 91/60891/12
Mohamed ElSerngawy [Fri, 28 Jul 2017 20:52:02 +0000 (16:52 -0400)]
Move idmlight bundle to aaa-shiro bundle

Moving idmlight to shiro bundle and adjust
odl-aaa-shiro feature dependenies.

Change-Id: I1d750a1012aa2e8d29e489e03994672c871dd784
Signed-off-by: Mohamed ElSerngawy <m.elserngawy@gmail.com>
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoFix checkstyle issues to enforce it 29/63229/1
David Suarez [Mon, 18 Sep 2017 12:24:20 +0000 (14:24 +0200)]
Fix checkstyle issues to enforce it

Change-Id: I467509334fb6c9c0d2be24a08b2d2f6eba0f3f4d
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
6 years agoFix checkstyle issues to enforce it 89/63089/5
David Suarez [Thu, 14 Sep 2017 15:29:44 +0000 (17:29 +0200)]
Fix checkstyle issues to enforce it

- Add an entry to .gitignore.

Change-Id: I03757a605211497210e7b7aa7b635ca65b69bb84
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>
6 years agoUtility service to encode/decode public/private keys of type RSA, DSA and ECDSA. 12/62712/2
Atul Gosain [Wed, 7 Jun 2017 22:10:02 +0000 (15:10 -0700)]
Utility service to encode/decode public/private keys of type RSA, DSA and ECDSA.
Placing it here to make a common place to access this service.

Change-Id: I614a49fb7adc8fe569fc2959a79ce4a74fe6b548
Signed-off-by: Atul Gosain <agosain@brocade.com>
6 years agoUpdate odlparent to 2.0.4 56/61856/2
Thanh Ha [Wed, 16 Aug 2017 14:59:36 +0000 (10:59 -0400)]
Update odlparent to 2.0.4

Also remove unused property karaf.resources.version.

Change-Id: I0eecbcd3f20cf0844a400151ede817064defc285
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
6 years agoBug 9040: avoid using dynamicAuthorization for cluster-admin operations 33/62233/1
Evan Zeller [Wed, 23 Aug 2017 22:24:40 +0000 (15:24 -0700)]
Bug 9040: avoid using dynamicAuthorization for cluster-admin operations

In some scenarios such as unexpected loss of voting members in a
geo-distributed cluster the cluster-admin RPC
change-member-voting-states-for-all-shards should be executed to convert
the member to voting. This RPC needs to execute regardless of if there
is a leader so we should avoid using the dynamicAuthorization via MD-SAL

Change-Id: I1c992d0da7a99171e131c7979ff4f22d740aaaed
Signed-off-by: Evan Zeller <evanrzeller@gmail.com>
6 years agoBump commons dependencies, clean up POM 44/61844/1
Stephen Kitt [Wed, 16 Aug 2017 11:48:23 +0000 (13:48 +0200)]
Bump commons dependencies, clean up POM

Explicitly pull in beanutils 1.9.3 in aaa-shiro, and clean up the
dependencies to pull in dependencies transitively. The generated
feature ends up being identical, apart from beanutils and
commons-logging (upgraded to 1.2, which is now an OSGi bundle).

beanutils’ release notes are available at
https://commons.apache.org/proper/commons-beanutils/changes-report.html

Change-Id: I4fe725c6d4b443e0c3be3cbdd662a27a7ca7c719
Signed-off-by: Stephen Kitt <skitt@redhat.com>
6 years agoBump versions by x.(y+1).z for next dev cycle 73/61673/1
Thanh Ha [Mon, 14 Aug 2017 17:14:34 +0000 (13:14 -0400)]
Bump versions by x.(y+1).z for next dev cycle

Change-Id: I244639b1a978de38bf88ef25f72a7c24f46da13c
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
6 years agoBump odlparent 2.0.2 to 2.0.4 66/61366/2
Stephen Kitt [Tue, 8 Aug 2017 16:21:02 +0000 (18:21 +0200)]
Bump odlparent 2.0.2 to 2.0.4

Change-Id: Iaddd898e2c40e284dd61b435a95064831ad88529
Signed-off-by: Stephen Kitt <skitt@redhat.com>
6 years agoUse current Odlparent version 83/60983/2
Vratko Polak [Tue, 1 Aug 2017 12:21:03 +0000 (14:21 +0200)]
Use current Odlparent version

Two properties were using 2.0.1, bumping to 2.0.2:
- karaf.resources.version
- features.test.version

Change-Id: I2a1cc9f102c78c73947acec2c31bc1a7bb8cfb95
Signed-off-by: Vratko Polak <vrpolak@cisco.com>
6 years agoBug 8915: Clean up context-path registration in AAA 57/60957/3
Ryan Goulding [Mon, 31 Jul 2017 18:06:30 +0000 (14:06 -0400)]
Bug 8915:  Clean up context-path registration in AAA

It was discovered that both aaa-shiro and dlux were trying to
take ownership of the root servlet context ("/").  Really, neither
probably should!  However, in order to maintain backwards compatibility,
AAA team registered the contexts individually through HttpService and
avoids taking ownership of root servlet context.  The long term solution
should involve migrating dlux off the "http://localhost:8181/index.html"
to something like "http://localhost:8181/dlux/index.html" since no single
bundle should probably own the root servlet context IMO.

Change-Id: I42b310fc54e5f7f7fe0d18e395bd3ee768fece18
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRealign idmlight endpoints to use AAAShiroFilter 28/59628/4
Ryan Goulding [Wed, 28 Jun 2017 16:46:25 +0000 (12:46 -0400)]
Realign idmlight endpoints to use AAAShiroFilter

Due to moving around servlet contexts and repurposing Servlet Filters,
it is time to realign idmlight to utilize AAAShiroFilter instead of
AAAFilter.  This designates that idmlight endpoints aaa control is
toggled independently of odl-restconf's.

Change-Id: I26402b6b5b9c482dd5eae8fefb4e6a63f7dae8c4
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoRemove derivation from controller config-parent 63/60663/1
Tom Pantelis [Sat, 22 Jul 2017 17:46:30 +0000 (13:46 -0400)]
Remove derivation from controller config-parent

aaa does not use the config subsystem so does not need to
derive from controller config-parent.

Change-Id: Ieab26416badc9c4a8b3871ab2edc8276f8cc7847
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
6 years agoClean odl-aaa-authn feature 73/60573/1
Mohamed ElSerngawy [Wed, 19 Jul 2017 19:31:30 +0000 (15:31 -0400)]
Clean odl-aaa-authn feature

Move all the dependencies from odl-aaa-authn feature
to odl-aaa-shiro feature in order to have one feature
file to maintaine and update.

Change-Id: I6e7781a24b9a071483456f97bbb9cde6ea267305
Signed-off-by: Mohamed ElSerngawy <m.elserngawy@gmail.com>
6 years agoDeprecate SHA256Calculator 26/60426/1
Ryan Goulding [Sat, 15 Jul 2017 13:07:50 +0000 (09:07 -0400)]
Deprecate SHA256Calculator

Shiro provides a much more robust encryption API that we should be using.
Deprecate this custom code in favor of that.

Change-Id: I2e506f0e9d4b29b23933730a45d08ddec0c53425
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
6 years agoBug 8721: CLI (standalone) with new "check password" feature 49/60049/4
Michael Vorburger [Thu, 6 Jul 2017 11:23:29 +0000 (13:23 +0200)]
Bug 8721: CLI (standalone) with new "check password" feature

Change-Id: I19ba495df60ebe08c71ce3c9c6cc24717b0f3856
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
6 years agoMerge "Export aaa-cli-jar in the artifacts"
Ryan Goulding [Fri, 14 Jul 2017 16:02:44 +0000 (16:02 +0000)]
Merge "Export aaa-cli-jar in the artifacts"

6 years agoFix hang in KarafIniWebEnvironment#init 09/60309/2
Tom Pantelis [Thu, 13 Jul 2017 20:49:17 +0000 (16:49 -0400)]
Fix hang in KarafIniWebEnvironment#init

It's possible the call to getInstance races with the BP container startup
in which case the INSTANCE will be created with all nulls and thus the
shiroConfiguration will forever be null. To get a valid INSTANCE, it must
be created via BP which injects a non-null ShiroConfiguration.

So getInstance now just returns INSTANCE and KarafIniWebEnvironment#init
busy waits on it.

Change-Id: I9c22d0dbec580c15830ece10c7dfa27f68ab8acf
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
6 years agoExport aaa-cli-jar in the artifacts 91/60291/1
Stephen Kitt [Thu, 13 Jul 2017 14:51:59 +0000 (16:51 +0200)]
Export aaa-cli-jar in the artifacts

aaa-cli-jar is used by dependent projects, add it to the artifacts.

Change-Id: I36da018c2c51aadf8422bb575f95f9c466ded30f
Signed-off-by: Stephen Kitt <skitt@redhat.com>
6 years agoUpgrade to odlparent 2.0.2 89/60089/1
Stephen Kitt [Fri, 7 Jul 2017 15:46:16 +0000 (17:46 +0200)]
Upgrade to odlparent 2.0.2

Change-Id: I667f30ce948774ab6f6c734e7072f48a16229616
Signed-off-by: Stephen Kitt <skitt@redhat.com>
6 years agoBump odlparent dependency to 2.0.1 56/59856/2
Robert Varga [Mon, 3 Jul 2017 09:02:05 +0000 (11:02 +0200)]
Bump odlparent dependency to 2.0.1

Bumps odlparent to latest release.

Change-Id: I52cd3772ee14f2bd3c2cb0fd4708ad1f23dde339
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
7 years agoMerge "Clean aaa features"
Mohamed El-Serngawy [Thu, 29 Jun 2017 13:46:26 +0000 (13:46 +0000)]
Merge "Clean aaa features"

7 years agoBug 7793: provide a script to convert shiro.ini to rest payload 88/59588/4
Ryan Goulding [Tue, 27 Jun 2017 19:54:48 +0000 (15:54 -0400)]
Bug 7793:  provide a script to convert shiro.ini to rest payload

Since most people have relied on shiro.ini functionality, it
is fitting that the AAA team supplies a script to help with
the transition to the clustered-app-config.

This script works as follows:
> python bin/upgrade/convert-shiro-ini-to-rest-payload <filename>

The resulting XML is output to stdout.

Change-Id: Ia2454a757eba4024290417588ca5e79b454ab072
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Bug 7793: shiro.ini needs to die"
Mohamed El-Serngawy [Wed, 28 Jun 2017 13:49:19 +0000 (13:49 +0000)]
Merge "Bug 7793: shiro.ini needs to die"

7 years agoClean aaa features 02/59502/2
Ryan Goulding [Sat, 24 Jun 2017 17:03:44 +0000 (13:03 -0400)]
Clean aaa features

Change-Id: Ic18a415e803ee8435b44f4f44aabbba20ee7db78
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBug 7793: shiro.ini needs to die 44/59244/10
Ryan Goulding [Tue, 20 Jun 2017 15:36:03 +0000 (11:36 -0400)]
Bug 7793: shiro.ini needs to die

shiro.ini is a poor means for configuration in our configuration-rich
ODL context.  Some primary concerns involve the fact that shiro.ini
is not replicated nor consistent across a cluster of ODL nodes.
Prior to this change, lack of proper SOA means that services couldn't
be dynamically decided at runtime (i.e., swap out backing
implementation).

Simply put, shiro.ini sucks.  This was a known limitation when shiro
was first integrated, but was swallowed hook line and sinker for the
other advantages that the integration provided.  It is time for us to
change this so that shiro is configured using a more mature and robust
mechanism (i.e., clustered-app-config).

Some known limitations in this patch include the fact that, because we
utilize jersey 1.X with its static configuration through web.xml, there
is a race between when the configuration is available, and when the
web service is instantiated.  To overcome this limitation, a basic
waiting mechanism was added to KarafIniWebEnvironment to wait until
the configuration is available prior to initializing Shiro.

The name "aaa-app-config" was chosen on purpose-- this is due to the
fact that most AAA related properties should eventually be configured
through this single pane solution.  This will reduce complexity and
reliance on running disparate ManagedService instances.

Change-Id: Ie87886dd57958957dfd0a642fec434862903b509
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBug 8717 Deprecate the existing non-functional mdsal impl 89/59189/2
Ryan Goulding [Mon, 19 Jun 2017 18:53:42 +0000 (14:53 -0400)]
Bug 8717 Deprecate the existing non-functional mdsal impl

The mdsal based backing datastore impl is broken, and utilizes
many unsavory practices including reflection left and right.
Deprecate it because it isn't currently used, so it can be
removed in Oxygen and replaced with a proper implementation
utilizing greatly simplified code.

Change-Id: I0a1fc0f6ff69c6d5d54b809740b558e46eb793b9
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBump odlparent version 99/59299/2
Robert Varga [Wed, 21 Jun 2017 15:19:44 +0000 (17:19 +0200)]
Bump odlparent version

Fixes the fallout from migration.

Change-Id: I5d10f0a8bb84638059a083617b7853362e669269
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
7 years agoBug 8661: Remove references to servicemix beanutils 48/59248/1
Ryan Goulding [Tue, 20 Jun 2017 16:55:12 +0000 (12:55 -0400)]
Bug 8661:  Remove references to servicemix beanutils

This was originally done in haste to get shiro in.  It was
silly and stupid, and is now removed :).

servicemix has a repackaged version of beanutils that isn't
needed;  we have the original one.  No need to push this in.

Sorry for the confusion.

Change-Id: Ief897e099c8fdded4e1cdd14c24865d1f95698cb
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBug 7265: Fix idmtool paths to reflect the path setup in Nitrogen 76/59176/1
Ryan Goulding [Mon, 19 Jun 2017 15:07:57 +0000 (11:07 -0400)]
Bug 7265: Fix idmtool paths to reflect the path setup in Nitrogen

In Nitrogen, idmtool was moved from etc to bin.  The paths for
related file access, however, were never updated.  This fixes
this oversight in two critical areas:
1) disabling SSL for idmtool
2) changing the jolokia password

Change-Id: I39554f866494a725638d5b9357d766d1b160695f
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBump OLTU 1.0.0 → 1.0.2 66/58966/2
Stephen Kitt [Wed, 14 Jun 2017 16:29:41 +0000 (18:29 +0200)]
Bump OLTU 1.0.0 → 1.0.2

Change-Id: I72780529787438b67e8d2393f631b194c60f56fd
Signed-off-by: Stephen Kitt <skitt@redhat.com>
7 years agoFinal removal of Activators in shiro and shiro-act 48/59048/1
Ryan Goulding [Thu, 15 Jun 2017 19:11:06 +0000 (15:11 -0400)]
Final removal of Activators in shiro and shiro-act

aaa-shiro and aaa-shiro-act are now completely wired using
blueprint instead of Activators or the configuration subsystem.

This patch fixes documentation and does the final removal of
the Activator.

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