aaa.git
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>
7 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>
7 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>
7 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"

7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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>
7 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"

7 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>
7 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>
7 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>
7 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>
7 years agoConvert aaa-shiro-act to blueprint 86/50586/6
Ryan Goulding [Tue, 17 Jan 2017 23:50:50 +0000 (18:50 -0500)]
Convert aaa-shiro-act to blueprint

Change-Id: I11cc228b7b584fa7220212d20e98c9ae3ae32fec
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBug 8437: Collapse Services and transition to blueprint 75/58275/15
Ryan Goulding [Mon, 5 Jun 2017 16:26:32 +0000 (12:26 -0400)]
Bug 8437: Collapse Services and transition to blueprint

One of the major focuses of the Nitrogen release surrounds service stability
in Karaf.  Hitherto, AAA has made use of a wide variety of service wiring
techniques, some effective, some not so great.  The service wiring really
was composed of three forms:
1) blueprint (good)
2) CSS (removed in previous patches and replaced by #1)
3) felix dependency management, which is really old and required long waits
   to properly wire various components

Since #2 was already removed, this patch removes #3, which is the wiring of
services through ServiceTrackerCustomizers and the singleton ServiceLocator.
ServiceLocator still exists in this patch;  to remove it here would make the
patch much too large, but it will be refactored in a followup patch.

For now, what is done is the following:
1) removal of aaa-authn bundle.  This bundle contained Builders and utility
classes common to aaa-idmlight and aaa-mdsal-store bundles.  They are
implementation details, and thus not candidates for odl-aaa-api.  They were
put in odl-aaa-shiro, as eventually odl-aaa-mdsal-cluster will eventually be
removed and replaced with functionality built directly into odl-aaa-shiro
(see https://git.opendaylight.org/gerrit/#/c/56744/)

2) removal of aaa-authn-basic.  This bundle really didn't provide much
more than HttpAuthBasic, which is really the only common impl of TokenAuth
used in AAA.  This bundle's functionality was relocated to odl-aaa-shiro.

3) removal of aaa-authn-sts.  STS stands for "single token store".  This
bundle also provided limited functionality, and really didn't make sense as
a completely separate service.

This patch focuses on the consolidation of several aaa bundles and services
into something more manageable directly in aaa-shiro.

Future work will:
1) remove the existing mdsal store in favor of the MDSAL based
   AuthorizingRealm.  Basically, the old mdsal store utilizes a lot of hacky
   code, reflection, etc. which can be cleaned and redefined to be more
   managable.  This will be removed in Oxygen, as we forgot to deprecate
   it in Nitrogen.  Messaging will be delivered to ensure that people use
   the new Realm approach once it is available.  This is again, not going
   to be the default store anyway in Nitrogen.

2) further refactoring of Services so that the aaa-idmlight and aaa-h2-store
   bundles are included directly in aaa-shiro.  Basically, it is really
   against OSGi architecture to have this many services providing such
   limited functionality.  Instead, we ought to provide them as part of a
   single service, and make configuration through clustered-app-config,
   cfg files, or some other more sane way than service injection.  The
   fact remains that we really want a more statically wired configuration
   that is consistent and easy to understand.

This is just part of many patches to clean the existing AAA implementation.

Change-Id: I740a667278f2ff64daff5e3602ddc5586fe23733
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoFix Remove Role command 83/58983/1
David Suarez [Wed, 14 Jun 2017 23:12:20 +0000 (01:12 +0200)]
Fix Remove Role command

The Remove Role command was actually removing the user with roleId
instead of the role.

Change-Id: I8564c35d658a6367dbf3e16eab23f46803119721
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoMove local karaf distribution to karaf4 77/58077/3
Ryan Goulding [Wed, 31 May 2017 19:11:18 +0000 (15:11 -0400)]
Move local karaf distribution to karaf4

Move the local karaf distribution to karaf4.  Also adds a
missing artifact to the artifacts pom.

Change-Id: I881794d3d3cdc22ed9776f87480b8c9437434e71
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Remove executable bit from XML files"
Mohamed El-Serngawy [Wed, 14 Jun 2017 12:56:54 +0000 (12:56 +0000)]
Merge "Remove executable bit from XML files"

7 years agoMerge "Migrate to odlparent 1.9.0"
Thanh Ha [Wed, 14 Jun 2017 03:50:04 +0000 (03:50 +0000)]
Merge "Migrate to odlparent 1.9.0"

7 years agoRemove executable bit from XML files 90/58890/1
Thanh Ha [Tue, 13 Jun 2017 20:38:44 +0000 (16:38 -0400)]
Remove executable bit from XML files

XML files are not executable so do not need the bit set.

Change-Id: I84cacd5ce2cdc9905df928a4d453e5cfe503db23
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
7 years agoAdd functionality to change jolokia password 30/58830/2
Ryan Goulding [Tue, 13 Jun 2017 15:48:07 +0000 (11:48 -0400)]
Add functionality to change jolokia password

Jolokia authentication is still unfortunately limited to
authMode=basic.  In the meantime, functionality is added
to allow easy change of this password through the exposed
jolokia config file org.jolokia.osgi.cfg.

In the future this password will be tied to AAA directly.
For now, just expose an easy utility to change it.

Change-Id: I53b7f67326dcc6dbd09a6e42b0f055ff8d1766d8
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMigrate to odlparent 1.9.0 47/58847/1
Thanh Ha [Tue, 13 Jun 2017 18:42:05 +0000 (14:42 -0400)]
Migrate to odlparent 1.9.0

Change-Id: I735d0de85bd282603c45caf95484ad1868c7eab4
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
7 years agoMerge "Revert "Refactoring AAA datastore bundles""
Ryan Goulding [Tue, 13 Jun 2017 15:25:08 +0000 (15:25 +0000)]
Merge "Revert "Refactoring AAA datastore bundles""

7 years agoRevert "Refactoring AAA datastore bundles" 19/58819/1
Ryan Goulding [Tue, 13 Jun 2017 13:04:33 +0000 (13:04 +0000)]
Revert "Refactoring AAA datastore bundles"

This reverts commit d00348582c23379e0f5baca555f378314d9bf560.

Change-Id: I574ab168e53b853b2a99737129dc3ddeb3025a6a
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Add gitignore to aaa-cert"
Ryan Goulding [Mon, 12 Jun 2017 20:11:07 +0000 (20:11 +0000)]
Merge "Add gitignore to aaa-cert"

7 years agoMerge "Refactoring AAA datastore bundles"
Ryan Goulding [Mon, 12 Jun 2017 19:23:25 +0000 (19:23 +0000)]
Merge "Refactoring AAA datastore bundles"

7 years agoAdd gitignore to aaa-cert 55/58755/1
Ryan Goulding [Mon, 12 Jun 2017 17:20:59 +0000 (13:20 -0400)]
Add gitignore to aaa-cert

Avoid some files that are laid down during compilation.

Change-Id: I4903e1f7ea56d235a834ba5caaffec6d8504c9fb
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoRefactoring AAA datastore bundles 08/57308/9
Mohamed El-Serngawy [Wed, 17 May 2017 20:57:18 +0000 (16:57 -0400)]
Refactoring AAA datastore bundles

The two datastore bundles aaa-mdsal and aaa-h2 are
combiend in the aaa-idmlight bundle. Also the aaa-features
bundle has been updated to match the new changes

Change-Id: Ic0413a1deb7704ed4f93c948d887dca92779b4e7
Signed-off-by: Mohamed El-Serngawy <serngawy@gmail.com>
7 years agoRemove idmlight.db.mv.db from source 18/58618/1
Ryan Goulding [Fri, 9 Jun 2017 14:44:18 +0000 (10:44 -0400)]
Remove idmlight.db.mv.db from source

Remove from source.  Accidental add.

Change-Id: I6831e3ac020980ca799082e771a44697221dbdd0
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMigrate to odlparent 1.8.0-Carbon 92/58292/1
Thanh Ha [Tue, 6 Jun 2017 01:42:26 +0000 (21:42 -0400)]
Migrate to odlparent 1.8.0-Carbon

Per request of odlparent project we are downgrading all Nitrogen
projects to use the released odlparent 1.8.0-Carbon to allow for the
odlparent project to start performing semver style releases.

Jira: RELENG-159
RT: 41406
Change-Id: Ia00b3a90aa68d66a05570be8eca13b6144ca70c2
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
7 years agoFix issues in checkstyle enforcement for module aaa-h2-store 23/57023/4
David Suarez [Sun, 14 May 2017 17:13:32 +0000 (19:13 +0200)]
Fix issues in checkstyle enforcement for module aaa-h2-store

- Copyright section
- Order imports
- Line lengths
- Remove IllegalCatch
- Improve exception handling
- Declare some static constants properly
- JavaDocs
- Move overloaded methods closer to each other
- Change variables and methods names
- Enforcement in pom.xml

Change-Id: If9ce15fdfcacaa7d94e56368cdf8c74b1b140ece
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoMerge "Fix issues in checkstyle enforcement for module aaa-cert"
Ryan Goulding [Mon, 5 Jun 2017 13:21:21 +0000 (13:21 +0000)]
Merge "Fix issues in checkstyle enforcement for module aaa-cert"

7 years agoMerge "Bug 8437: remove deprecated aaa-authn-store bundle"
Ryan Goulding [Mon, 5 Jun 2017 13:20:55 +0000 (13:20 +0000)]
Merge "Bug 8437: remove deprecated aaa-authn-store bundle"

7 years agoFix issues in checkstyle enforcement for module aaa-cert 01/57001/7
David Suarez [Sat, 13 May 2017 20:14:39 +0000 (22:14 +0200)]
Fix issues in checkstyle enforcement for module aaa-cert

- Copyright section
- Line lengths
- JavaDocs
- Order imports
- Declare some static constants properly
- Move overloaded methods closer to each other
- Returning variables directly
- Change variables and methods names
- Enforcement in pom.xml

Change-Id: I15fffbeb9de5f08e10aebd640437ddbbc97cf94b
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoFix issues in checkstyle enforcement for module aaa-idmlight 24/57024/4
David Suarez [Sun, 14 May 2017 18:55:44 +0000 (20:55 +0200)]
Fix issues in checkstyle enforcement for module aaa-idmlight

- Copyright section
- Order imports
- Line lengths
- JavaDocs
- Improve exception handling
- Change variables and methods names
- Move variables closer to their use
- Move overloaded methods closer to each other
- Declare some static constants properly
- Enforcement in pom.xml

Change-Id: Ic673b8239c3abd7ba0c23a242293fe32cc48b103
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoMerge "Fix issues in checkstyle enforcement for module aaa-cli"
Mohamed El-Serngawy [Fri, 2 Jun 2017 16:00:08 +0000 (16:00 +0000)]
Merge "Fix issues in checkstyle enforcement for module aaa-cli"

7 years agoBug 8437: remove deprecated aaa-authn-store bundle 41/58141/1
Ryan Goulding [Thu, 1 Jun 2017 21:00:34 +0000 (17:00 -0400)]
Bug 8437: remove deprecated aaa-authn-store bundle

This was deprecated long ago.  This is the removal of the bundle.

Change-Id: I42caef0fe749a7d52a1e0cbf8a2440bbaf93d548
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBug 8437: Move oauth2 functionality into the shiro module 36/58136/1
Ryan Goulding [Thu, 1 Jun 2017 19:12:53 +0000 (15:12 -0400)]
Bug 8437: Move oauth2 functionality into the shiro module

odl-aaa-shiro is the only consumer of the oauth2 functionality in ODL.
The fact that aaa-authn-sts is aggregated as a separate bundle for
this functionality is confusing and superfluous.  This change moves
the functionality, does some good renaming, and puts in some docs
surrounding the current status of aaa-authn-sts and aaa-shiro.

This change was tested manually with odl-restconf and works
correctly.

In the future, in a follow up, aaa-authn-sts will be completely
removed.  For now it is still keeping track of the various "services"
that we are working to converge.

Change-Id: Id9cc5bd70e0866d72b712ddc559c1cd0154268d9
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Fix a gitignore and accidentally added file"
Ryan Goulding [Thu, 1 Jun 2017 17:50:40 +0000 (17:50 +0000)]
Merge "Fix a gitignore and accidentally added file"

7 years agoFix a gitignore and accidentally added file 23/58123/1
Ryan Goulding [Thu, 1 Jun 2017 16:20:46 +0000 (12:20 -0400)]
Fix a gitignore and accidentally added file

Change-Id: I15bd3190baa5298d953e2cfa52ed8dfed13b8418
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoFix issues in checkstyle enforcement for module aaa-cli 02/57002/3
David Suarez [Sun, 14 May 2017 00:27:12 +0000 (02:27 +0200)]
Fix issues in checkstyle enforcement for module aaa-cli

- Copyright section
- Line lengths
- JavaDocs
- Order imports
- Declare some static constants properly
- Remove IllegalCatch
- Change variables and methods names
- Enforcement in pom.xml

- Move overloaded methods closer to each other
- Returning variables directly

Change-Id: I4bd93465b43610cfd931b104803142b371c42240
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoRemove unused UnauthorizedException 81/58081/1
Ryan Goulding [Wed, 31 May 2017 22:42:18 +0000 (18:42 -0400)]
Remove unused UnauthorizedException

This is not used, and should be removed.

Change-Id: I3e9390df44652d153565f97dab8f390f2b7339f3
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Bug 8572: Remove unused SecureBlockingQueue"
Ryan Goulding [Wed, 31 May 2017 22:18:06 +0000 (22:18 +0000)]
Merge "Bug 8572: Remove unused SecureBlockingQueue"

7 years agoMerge "Fix issues in checkstyle enforcement for module aaa-shiro-api"
Ryan Goulding [Wed, 31 May 2017 21:48:50 +0000 (21:48 +0000)]
Merge "Fix issues in checkstyle enforcement for module aaa-shiro-api"

7 years agoBug 8572: Remove unused SecureBlockingQueue 80/58080/2
Ryan Goulding [Wed, 31 May 2017 21:23:56 +0000 (17:23 -0400)]
Bug 8572: Remove unused SecureBlockingQueue

This is unused and needs to be removed.

Change-Id: I141af03f6852bb69fbc8518eb7cf10e685cebca4
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoFix a few formatting issues 31/57831/1
Ryan Goulding [Thu, 25 May 2017 19:40:33 +0000 (15:40 -0400)]
Fix a few formatting issues

https://git.opendaylight.org/gerrit/#/c/57826/ intro'd
a few poor formatting issues.

Change-Id: Ib04c2899a3de4c03a5dfadbd45b652f32ecf1635
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoBug 8352: Explicitly set the SecurityManager 26/57826/1
Ryan Goulding [Thu, 25 May 2017 17:42:51 +0000 (13:42 -0400)]
Bug 8352: Explicitly set the SecurityManager

This issue is described more in depth in this thread:

http://shiro-user.582556.n2.nabble.com/ \
Shiro-with-Wicket-on-OSGi-ServiceMix-td7580484.html

We found that sometimes the SecurityManager for Shiro
is not set otherwise

Change-Id: I69829136f999d458058f0f112240cdf29bd1edc5
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoFix a parameter description in the CLI 01/57701/1
David Suarez [Tue, 23 May 2017 14:12:57 +0000 (16:12 +0200)]
Fix a parameter description in the CLI

Minor fix for the user's email parameter description.

Change-Id: Iae54cebbba72ce315eaca3928ee822f401c08210
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoMerge "Bug 8382: Update idmtool to not utilize ids in payload"
Mohamed El-Serngawy [Tue, 16 May 2017 13:41:25 +0000 (13:41 +0000)]
Merge "Bug 8382: Update idmtool to not utilize ids in payload"

7 years agoBug 8382: Update idmtool to not utilize ids in payload 93/57093/1
Ryan Goulding [Mon, 15 May 2017 18:08:56 +0000 (14:08 -0400)]
Bug 8382: Update idmtool to not utilize ids in payload

id is an internal implementation detail, as outlined by
bug 8382.  Prior patches were submitted to enforce that.
This patch just removes specification of the id as part
of the wrapped requests.

Change-Id: If1e12b899840b2c316400846bfcaf3c91d1ec11d
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Remove federation related documents"
Mohamed El-Serngawy [Mon, 15 May 2017 17:47:18 +0000 (17:47 +0000)]
Merge "Remove federation related documents"

7 years agoMerge "Update idmtool for oauth2 token generation"
Mohamed El-Serngawy [Mon, 15 May 2017 17:45:33 +0000 (17:45 +0000)]
Merge "Update idmtool for oauth2 token generation"

7 years agoMerge "Fix checkstyle issues in module aaa-authn-mdsal-store-impl"
Mohamed El-Serngawy [Mon, 15 May 2017 17:44:22 +0000 (17:44 +0000)]
Merge "Fix checkstyle issues in module aaa-authn-mdsal-store-impl"

7 years agoMerge "Fix issues in checkstyle enforcement for module aaa-encrypt-service"
Mohamed El-Serngawy [Mon, 15 May 2017 17:30:04 +0000 (17:30 +0000)]
Merge "Fix issues in checkstyle enforcement for module aaa-encrypt-service"

7 years agoMerge "Fix issues in checkstyle enforcement for module aaa-auth-mdsal-api"
Mohamed El-Serngawy [Mon, 15 May 2017 17:00:24 +0000 (17:00 +0000)]
Merge "Fix issues in checkstyle enforcement for module aaa-auth-mdsal-api"

7 years agoUpdate idmtool for oauth2 token generation 87/57087/1
Ryan Goulding [Mon, 15 May 2017 16:41:14 +0000 (12:41 -0400)]
Update idmtool for oauth2 token generation

Add functionality to grab an oauth2 token using idmtool.

Change-Id: I3b1ccd8a27ed01dbbf4f5388421c14598c64d89c
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoFix issues in checkstyle enforcement for module aaa-shiro-api 25/57025/1
David Suarez [Sun, 14 May 2017 19:23:52 +0000 (21:23 +0200)]
Fix issues in checkstyle enforcement for module aaa-shiro-api

- Enforcement in pom.xml

Change-Id: I4b0e3a696b8f1b8b5486fd5500f960126cd6887f
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoFix issues in checkstyle enforcement for module aaa-encrypt-service 05/57005/1
David Suarez [Sun, 14 May 2017 11:05:56 +0000 (13:05 +0200)]
Fix issues in checkstyle enforcement for module aaa-encrypt-service

- Copyright section
- Remove redundant modifiers
- Move overloaded methods closer to each other
- Order imports
- Line lengths
- Declare some static constants properly
- Change variables and methods names
- JavaDocs
- Enforcement in pom.xml

Change-Id: Ie9e5031e3b1fa0854e16eaab6798f4c7b310bd73
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoRemove aaa-credential-store-api 80/56980/2
Ryan Goulding [Fri, 12 May 2017 17:30:25 +0000 (13:30 -0400)]
Remove aaa-credential-store-api

This was deprecated long ago.  It is time to remove it.

Change-Id: Iaa3ca9ca8eb4fc806f5aa2b93d10c1bb9bcddb0e
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoFix checkstyle issues in module 96/56996/1
David Suarez [Sat, 13 May 2017 17:08:08 +0000 (19:08 +0200)]
Fix checkstyle issues in module
aaa-authn-mdsal-store-impl

- Copyright sections
- Remove @SuppressWarnings("checkstyle:IllegalCatch")
  by fixing exception handling
- Enforcement in pom.xml

Change-Id: I9564db1b11f2166551cb4b8fbd627fd453ab71fa
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoFix issues in checkstyle enforcement for module aaa-auth-mdsal-api 95/56995/2
David Suarez [Sat, 13 May 2017 15:39:53 +0000 (17:39 +0200)]
Fix issues in checkstyle enforcement for module aaa-auth-mdsal-api

- Copyright section
- Enforcement in pom.xml

Change-Id: I7db3689e20e2d05e780bb9c4de4ddcdc1946b290
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoRemove federation related documents 79/56979/1
Ryan Goulding [Fri, 12 May 2017 16:58:44 +0000 (12:58 -0400)]
Remove federation related documents

Federation is now done through the Shiro based impl.  The
docs related to the removed odl-aaa-authn-federation
functionality are no longer needed.

Change-Id: I9a635a3e1630a9e7bffc5e3bb8ac704faad97389
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoRemove <prerequisites><maven> from pom.xml to avoid WARNING 15/56815/1
David Suarez [Wed, 10 May 2017 23:01:39 +0000 (01:01 +0200)]
Remove <prerequisites><maven> from pom.xml to avoid WARNING

[WARNING] The project ... uses prerequisites which is only intended for
maven-plugin projects but not for non maven-plugin projects. For such
purposes you should use the maven-enforcer-plugin. See
https://maven.apache.org/enforcer/enforcer-rules/requireMavenVersion.html

This is already done in odlparent.

Change-Id: Iedfaeb92fba3bf392554ece1f20358060c398a00
Signed-off-by: David Suarez <david.suarez.fuentes@ericsson.com>
7 years agoRemove aaa-idp-mapping bundle 35/56335/6
Ryan Goulding [Mon, 1 May 2017 20:02:10 +0000 (16:02 -0400)]
Remove aaa-idp-mapping bundle

Was deprecated in Boron, dissuaded in Carbon, now
removed in nitrogen.

Change-Id: I0f564d4b2806259eb00695dd194dc65bf394f99d
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoRemove sssd feature 39/56639/1
Ryan Goulding [Sat, 6 May 2017 16:16:47 +0000 (12:16 -0400)]
Remove sssd feature

Change-Id: Ie96dee58d75a6ede910cd37494672032415c3086
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "sssd functionality removal"
Mohamed El-Serngawy [Fri, 5 May 2017 18:10:42 +0000 (18:10 +0000)]
Merge "sssd functionality removal"

7 years agoMerge "Bug 7090: Move idmlight.db to the data directory"
Mohamed El-Serngawy [Fri, 5 May 2017 18:09:52 +0000 (18:09 +0000)]
Merge "Bug 7090: Move idmlight.db to the data directory"

7 years agoMerge "Bug 8383: no email in user creation results in NPE"
Mohamed El-Serngawy [Fri, 5 May 2017 18:09:12 +0000 (18:09 +0000)]
Merge "Bug 8383: no email in user creation results in NPE"

7 years agoBug 8382: disallow client specified ids 07/56607/1
Ryan Goulding [Fri, 5 May 2017 16:27:52 +0000 (12:27 -0400)]
Bug 8382: disallow client specified ids

ids are an implementation specific internal detail.  Thus,
AAA should endpoints should prevent users from specifying
such values.

This change fixes the behavior by returning 400 when ids
are specified.  Appropriate test cases are added.

Change-Id: I15cc66f9881347334b919dbeb6bc686694367bd6
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Remove the aaa-authn-federation bundle artifacts"
Ryan Goulding [Thu, 4 May 2017 20:43:11 +0000 (20:43 +0000)]
Merge "Remove the aaa-authn-federation bundle artifacts"

7 years agoBug 8383: no email in user creation results in NPE 58/56558/1
Ryan Goulding [Thu, 4 May 2017 20:32:20 +0000 (16:32 -0400)]
Bug 8383: no email in user creation results in NPE

Change-Id: I1eac566bcbfa0da44e09d7062e174fd597ccd0f4
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoMerge "Bug 8379: Remove deprecated TokenAuthFilter"
Mohamed El-Serngawy [Thu, 4 May 2017 20:25:30 +0000 (20:25 +0000)]
Merge "Bug 8379: Remove deprecated TokenAuthFilter"

7 years agoBug 7090: Move idmlight.db to the data directory 37/56337/2
Ryan Goulding [Mon, 1 May 2017 20:15:06 +0000 (16:15 -0400)]
Bug 7090: Move idmlight.db to the data directory

Simply change the default behavior to plant idmlight.db
in the data directory

Change-Id: Ic766d0306b92cbe87533af6c885ff08ca8e6c78c
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoRemove the aaa-authn-federation bundle artifacts 55/56555/1
Ryan Goulding [Thu, 4 May 2017 19:18:10 +0000 (15:18 -0400)]
Remove the aaa-authn-federation bundle artifacts

Change-Id: I55792e209755adf98f7190884404d253e8b23012
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agosssd functionality removal 54/56554/2
Ryan Goulding [Thu, 4 May 2017 19:12:40 +0000 (15:12 -0400)]
sssd functionality removal

odl-authn-sssd was deprecated in Boron.  The feature was removed
in Carbon, but the underlying bundles were kept just in case
(i.e., one could install manually but it wasn't a present
feature).  This completes the removal since adequate time has
been given for consumers to move on to the new method, Shiro.

Change-Id: Idb95c0f41a3db0300fd8fbd7272ec3ab842f9c45
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
7 years agoDo not override versions defined in odlparent 50/56550/1
Robert Varga [Thu, 4 May 2017 18:21:58 +0000 (20:21 +0200)]
Do not override versions defined in odlparent

This fixes commons-codec version duplication, cutting ~250kB
from distribution package.

Change-Id: I7812e5ac8ae02de25d9359eda12e78642cf33f41
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
(cherry picked from commit 457bafe812c6330ae314dd048e3e64ddd4165a68)

7 years agoBug 8379: Remove deprecated TokenAuthFilter 47/56547/2
Ryan Goulding [Thu, 4 May 2017 17:41:09 +0000 (13:41 -0400)]
Bug 8379: Remove deprecated TokenAuthFilter

TokenAuthFilter was deprecated in either Beryllium or Boron.
It has been kept around for compatibility reasons, but it
is no longer needed.  This patch removes the Filter class,
moves the inner UnauthorizedException to its own class
(better anyway), and removes some dead tests that were
testing the old filter functionality.  It is clear that
the tests were testing the old Filter mechanism instead
of AAAShiroFilter, since AUTH_FILTERS was set to
TokenAuthFilter.class.getName().  Thus, the test is no
longer appropriate.

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