Remove deprecated SHA256Calculator

Not used anywhere.

Change-Id: I1bfa0db35eb3e692cc1c98cf55b3c8ed004d3deb
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Remove deprecated StoreBuilder#init

Change-Id: I9f4a92572b0db28c9a07ad596a7b9f7cf92cf841
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Remove deprecated IdmLightConfig#getDbPath

Change-Id: I00e1b1143a6d4c38f42f1d9d26a6cce359a884ed
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Remove static CustomFilterAdapterConfigurationImpl instance

This was kept for backwards compatibility for web.xml's which
have been removed.

Change-Id: I9a7269a4807bf54fa3945c28ee78ad32a1725f24
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix some deprecation warnings

CheckedFuture et al.

Change-Id: Ibd4324fbc57367eb1ab1508bc56053977fb5e47a
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Remove KarafIniWebEnvironment

Deprecated and not used anymore.

Change-Id: Id06ffe5768c8564a97c9c7c08db1cdd622b82e10
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

adjust to use password-service

Use the simplified password-service instead of SHA256Calculator.
After all, SHA256Calculator is deprecated since it combines API
and IMPL even in the name!

This is also more configurable and secure.

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

Merge "Split aaa-encryption-service api and impl"

Merge "Convert to jersey 2"

Split aaa-encryption-service api and impl

The original contributor jammed api and impl into one bundle.  This is
bad practice for SOA, so this change separates out api and impl.  The
API class package remains the same for outside consumers (i.e., it is
left as org.opendaylight.aaa.encrypt instead of the normal convention
of org.opendaylight.aaa.encrypt.api).

Additionally, a maven-bundle-plugin instruction was added to explicitly
not export any impl bundle classes.  This is important to avoid possible
consumption from downstream consumers.

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

add .apt_generated_tests/ to .gitignore

Change-Id: Ie2c3be4911874cf59d7f691d357e45f03bdfee71
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Convert to jersey 2

Modified the WenInitializer to use the new servlet API and changed
the jersey client version to 2.25.1. Also modified the UTs to jersey 2
test framework.

JIRA: TSC-113

Change-Id: I3864bd217126954e93308699e095f67afc2e53da
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix STF error in odl-aaa-password-service

[caused by: Unable to resolve org.opendaylight.aaa.password-service-api/0.8.0.SNAPSHOT: missing requirement [org.opendaylight.aaa.password-service-api/0.8.0.SNAPSHOT] osgi.wiring.package; filter:="(&(osgi.wiring.package=org.opendaylight.yangtools.concepts)(version>=2.0.0)(!(version>=3.0.0)))"]]]

The feature needs to install the mdsal binding and yangtools
artifacts.

Change-Id: I1c98bc77e76d85181b559f499a6cfe38c0da4a7b
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Adjust to RPC method signature update

Input/Output structures are always present and we need to return
ListenableFuture.

Change-Id: Icce2f1091577d8741baf1bfd3b3de27463ca399e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Deprecate Encryption Service Impl

The default encryption service implemention is a mess that shouldn't be
maintained.  Instead, AAA team plans to add a new implementation that
is backed by Shiro cryptography.  We will expose a knob to control which
implementation is used at runtime.

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

Add odl-aaa-password-service feature

Add in a feature for use at runtime.

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

password service implementation

Implement PasswordHashService with a Default impl.  This impl is capable
of deriving values from aaa-password-service-config.yang.

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

password-service api cleanup

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

salt creation and password hash comparison service api

A generic one-way password comparison (hash equality) and salt generation
API.  This is not meant to cover Password criteria satisfaction.

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

Remove AAAFilter and aaa-shiro-act

There's no more usasges of AAAFilter and restcong no longer needs
aaa-shiro-act so remove them.

Change-Id: Ia763ee7f872b13d138ad49d6120495843a447599
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Add resource registration to web API

In order to access html files, jsps etc, the resource path(s) in the
bundle need to be registered with pax web. Added a resources property
to WebContext to capture this.

Change-Id: Ic47558588601cb340ab5b0c3c218fe43226ce769
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Add web/servlet-api and jersey2 implementation

web/servlet-api provides implementation-agnostic entrypoints for
creating Clients and HttpServlets.

web/servlet-jersey2 provides an implementation based on jersey-2.25.1.

This split allows us to have applications independent of the
implementation.

Change-Id: I77d92fb8764aa28817d5dcac9f8450dc42017429
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

add full implementation in web-jetty-impl

Change-Id: I649336bcaf51f683e52284cc549332c4c1815836
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

add skeleton web-jetty-impl (to be implemented)

This lets one use the new WebServer API outside of OSGi, e.g. in
component tests; specifically, I would like to use this in project
Neutron.

Change-Id: I7035078b877daaebceeb71a5e664386f6a85969a
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Align with odlparent version of jolokia

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

Ensure Jersey is initialised before AAA-Shiro

When we group the Jersey bundles with bundles using them, we run into
initialisation races where AAA-Shiro ends up trying to use Jersey
before the latter’s activators have run.

All credit to Robert Varga for figuring out that we need an ordering
constraint between Jersey as a whole and the rest of AAA-Shiro. The
new odl-aaa-jersey-1 feature will eventually be replaced by ODL
Parent’s odl-jersey-1 feature, once we’ve added jersey-client to
that.

Issue: RELENG-85
Change-Id: I3d87dc28c8067bbeb0ca32be96ccdb4f6d359573
Signed-off-by: Stephen Kitt <skitt@redhat.com>

Remove  javax.ws.rs-api dependency

Let's not pull javax.ws.rs-api-2.0.1 and see what gives.

Change-Id: I7c8656f4423e87818c844f49019f83fe39731bc4
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Align pax-web-api with Karaf 4.1.5

Karaf now uses version 6.0.9.

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

Bump to odlparent 3.1.0 and yangtools 2.0.3

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

Clean up odl-aaa-web

This needs Guava, so use odl-guava-23.

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

Clean up odl-aaa-shiro

Pull in odl-jolokia and odl-aaa-web to reduce the bundle overlap.

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

Add Karaf build profile

Project local Karaf distros are handy for devs to test their work
however is unneeded by autorelease builds and should not be released
as part of the Simultanious Release. Add a profile that is active by
default so that default behaviour is unchanged however allows the
autorelease project to disable building this module.

Change-Id: If26f62fd722bedce8d39d3dfe673064441fd1d36
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>
(cherry picked from commit 320971a7892e4540bc5d253cf9a2f8117b61e2ce)

AAA-143:  Remove jackson dependencies

Other projects need to pull in jackson runtime dependencies themselves
instead of depending on AAA.  AAA does not utilize Jackson anymore
period.

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

Remove aaa-filterchain Activator and statics

Removed the bundle Activator in lieu of blueprint and also
removed the static CustomFilterAdapterConfiguration instance.
CustomFilterAdapterConfiguration was converted to an interface
with implementation CustomFilterAdapterConfigurationImpl so it
can be advertised as a service and consumed by aaa-shiro and
injected into the CustomFilterAdapter.

Change-Id: Id1b6be949d9ce1bb895050e1ed95f321cdd2188a
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Use odl:type="default" for IdmLightProxy service reg

Change-Id: Ieb5d096aa64836e71ae6c1c7be810a36d49a907e
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Merge "remove Import-Package from aaa-shiro POM"

Merge "Convert IdmLightProxy CLAIM_CACHE to non-static"

Package aaa-shiro-act

This provides simple packaging of aaa-shiro-act, so netconf does
not have to package it itself.

JIRA: AAA-164
Change-Id: I4e65d102d15a0c35b579837840f9f46ae7ece7dd
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Convert IdmLightProxy CLAIM_CACHE to non-static

The map was static so the clearClaimCache method could be accessed
statically by UserHandler etc. Now the IdmLightProxy instance is injected
and referenced as a new interface, ClaimCache.

Change-Id: I7ed214c6158d950dc7da81813ca6b230dc3a6767
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Merge "introduce WebContextSecurer service API"

remove Import-Package from aaa-shiro POM

as far as I can tell from a quick test, it still works.

Change-Id: Id223170832378bed19f62e620f7353fb79723a74
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge changes from topic 'java-8-migration'

* changes:
  Java 8 migration
  Java 7 migration

Merge "Java 5 migration"

Java 8 migration

As suggested by IntelliJ:
* clean up lambdas;
* use new Map methods.

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

Merge "Remove unused code"

Remove unused code

Removed unused code.

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

Java 7 migration

As suggested by IntelliJ:
* remove redundant type specifiers;
* use try-with-resources.

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

Java 5 migration

As suggested by IntelliJ:
* use foreach loops;
* use StringBuilder instead of StringBuffer;
* drop unnecessary boxing constructors.

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

Add domain to the PasswordCredentials equality check

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

Remove EqualUtil

This patch uses Objects.equals() instead. The equality checks are
preserved as-is.

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

introduce odl-aaa-web feature

Change-Id: I3993ddd82e09d0075e47000b7ff75632b2bd5b3d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Merge changes I6062ddfa,If91c0ea5,Idd92e1be,I224e0fb7,Iab290548, ...

* changes:
  Enable findbugs in aaa-parent
   Fix findbugs violations in aaa-cli
  Fix findbugs violations in aaa-filterchain
  Fix findbugs violations in aaa-shiro
  Fix findbugs violations in aaa-cert
  Fix findbugs violations in aaa-encrypt-service

Merge changes Ia7a47d3b,I0d9b6fc2

* changes:
  Fix findbugs violations in aaa-authn-api
  Move checkstyle config to aaa-parent

introduce WebContextSecurer service API

This API allows other projects to secure their web context, but without
directly relying on AAA Shiro internals.  Using this, other applications
will be able to significantly reduce their dependencies, Package-Import
etc. to AAA Shiro internals.  (This opens the door both to more
independently evolve aaa-shiro internals, and allows for possible
alternative implementations, later.)

This also makes aaa-shiro secure its own IdmLightApplication REST
endpoints using the same approach, which avoids copy/paste of the
AAAShiroFilter and the KarafIniWebEnvironmentLoaderListener it needs
between the WebInitializer and the ShiroWebContextSecurer.

Change-Id: Ia3a16df71384610a75acf3d28205c973c554d477
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

ditch HashCodeUtil, and use JDK Objects.hash() instead

having a utility like this in Guava and in the JDK is probably are 1 too
much, let us not have another one doing the exact same thing in AAA as
well.

Change-Id: Icb19d3e5aed73eb46dee1394be0ae06181ab6ef4
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Enable findbugs in aaa-parent

Change-Id: I6062ddfa44de6cba7540beea5fbb8d215d3ca2d1
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

 Fix findbugs violations in aaa-cli

- Method may fail to close stream
- Reliance on default encoding
- Incorrect lazy initialization of static field
- Unread field: should this field be static?
- Write to static field from instance method

Change-Id: If91c0ea5997490468d030cab3aead2825fbe9c9e
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in aaa-authn-api

- Equals method should not assume anything about the type of its argument
- Reliance on default encoding
- Dead store to local variable
- Possible null pointer dereference on branch that might be infeasible
- Field not initialized in constructor but dereferenced without null check

Change-Id: Ia7a47d3b3b6a9729263c7c42656f14791edefccc
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in aaa-filterchain

- May expose internal representation by returning reference to mutable object
- Inefficient use of keySet iterator instead of entrySet iterator
- Field not initialized in constructor but dereferenced without null check

Change-Id: Idd92e1beb6998a6968ae6be3b5f1e83ae1ca50d7
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Move checkstyle config to aaa-parent

Change-Id: I0d9b6fc2f2eec27f2d438148bd3cb148901d72ff
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in aaa-shiro

- Possible null pointer dereference
- Class names shouldn't shadow simple name of implemented interface
- Method may fail to close database resource
- Non-transient non-serializable instance field in serializable class
- Non-serializable class has a serializable inner class
- Class is Serializable, but doesn't define serialVersionUID
- Consider using Locale parameterized version of invoked method
- Reliance on default encoding
- May expose internal representation by returning reference to mutable object
- Method invokes toString() method on a String
- Private method is never called
- Unread field
- Nonconstant string passed to execute or addBatch method on an SQL statement
- Unchecked/unconfirmed cast
- Dead store to local variable
- Class implements same interface as superclass
- Redundant nullcheck of value known to be non-null
- Exception is caught when Exception is not thrown
- Useless control flow

Change-Id: I224e0fb71f3570f69fa1963e89b8c687a464156a
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in aaa-cert

- Null pointer dereference
- Method ignores exceptional return value
- Method ignores results of InputStream.read()
- Method may fail to clean up stream or resource
- Method may fail to close stream on exception
- Reliance on default encoding
- Consider returning a zero length array rather than null
- Redundant nullcheck of value known to be non-null
- Potentially dangerous use of non-short-circuit logic

Change-Id: Iab2905488bbe2d4b9be3e92c69e49e5eb0129958
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix findbugs violations in aaa-encrypt-service

- Method may fail to clean up stream or resource
- Reliance on default encoding
- Method invokes inefficient new String(String) constructor
- Unchecked/unconfirmed cast

Change-Id: I0dd13b306a684167bacdf94648369150f365d590
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Derive all code sub-projects from aaa-parent

We can then centralize configs for CS and findbugs etc.

Change-Id: Iecca472fb7de14b34cf88b34765f7741d4e3c60b
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Fix checkstyle issues to enforce it

Change-Id: I77b3e119c7cd972f1f2f141f5adfdeab6c518ead
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

Remove static AuthenticationManager instance

It's only used by UT's.

Change-Id: I25271cd06d578942b7cf9cd35a38a338c5527f29
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Remove ServiceLocator

Removed the static instance holders in favor of injection.

Change-Id: Iea7beda16450f28af4119995da4768e931086592
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

Merge "Eliminate injection of AAAShiroProvider"

Merge "New shiro EnvironmentLoaderListener"

Eliminate injection of AAAShiroProvider

AAAShiroProvider is used as a holder for some instances and is injected
into other components just to access those instances. It's better to
directly inject the instances instead of having a dependency on
AAAShiroProvider.

Change-Id: Iaed51ae360360b3460c419eb4be2d4ffe3fdf558
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

New shiro EnvironmentLoaderListener

Added ShiroWebEnvironmentLoaderListener and AAAIniWebEnvironment
that inject the required instances instead of obtaining statically.

Change-Id: I5979342b7463a3634e9208eb813f32174c2a4cb4
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

make PaxWebServer ServiceFactory fail instead of return bogus WebServer

The current implementation may return a bogus defunct noop WebServer
which just ignores Servlet & Filter registrations (and just logs a WARN,
which could easily be overlooked) in case it cannot find the Pax Web
WebContainer service.

This change makes it instead "fail fast" on the WebServer service look
up.  Note that the ServiceFactory doc explicitly allows throwing
exceptions; that will be caught and turn into a null service reference,
which whatever is trying to obtain the WebServer must gracefully handle.

Change-Id: Ie4fcbdb125095d353d466958fade98fc759aefd4
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Refactor AAAShiroProvider & Co. to be non static

- The IdmLightApplication is now instantiated and injected
  with the AAAShiroProvider and passed to the ServletContainer
  instead of the ServletContainer instantiating it via reflection.

- For KarafIniWebEnvironmentLoaderListener and KarafIniWebEnvironment,
  the initial plan was to inject the AAAShiroProvider however there
  are still web.xml files in ODL land that reference
  KarafIniWebEnvironment and expect a no-arg ctor. We need to keep
  backwards compatibility for a while so I'll follow-up later
  to add a new KarafIniWebEnvironmentLoaderListener that is advertised
  as a service for programmtic use. KarafIniWebEnvironment was changed
  to obtain the ShiroConfiguration statically rather than the
  AAAShiroProvider.

- The shiro lib still instantiates the filter/realm etc instances via
  reflection. These are specified via String key/value pairs with class names
  in the Ini instance. Unfortunately I see no way around this. So
  to avoid having to pass our services, eg DataBroker, via statics,
  I opted to use ThreadLocals to inject indirectly. This is a bit
  ugly but works.

Change-Id: I8f5114802c76cbd2b4bfda69952df2b28557cf8d
Signed-off-by: Michael Vorburger <vorburger@redhat.com>
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

use web-api as dependency instead of web-osgi-impl in aaa-shiro

and amend @author in PaxWebServer for credit where credit is due

and some minor logging related clean up in PaxWebServer

Change-Id: Ibc4f4d8fd95f5d5693c11abcad4735af2c3e4d27
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

AAA-169: Advertise PaxWebServer as an OSGi service

The Pax Web WebContainer implementation registers a ServiceFactory and
uses the class loader of the bundle that obtains the OSGi service
reference. When PaxWebServer is advertised as a service, it causes a
ClassNotFoundEx when initializing shiro b/c it uses the TCCL that is
set by Pax Web obtained from the PaxWebServer's bundle. To alleviate this,
we can advertise a WebServer ServiceFactory so it use the caller's
bundle to get the WebContainer service.

Change-Id: I591c340ccb0551a8138d07ec79443bc648218baf
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

replace AAA's web.xml with programmatic registration

This is a first step with a like-for-like transformation;
future changes could go further; notably integrate it with
AAAShiroProvider which, strangely, had separate web registration
not using web.xml, and -likely- (TBC) replace the static
CompletableFuture "hoop" in AAAShiroProvider with normal DI.

Change-Id: I43c5fe90a087e2fbc68f779655c211253775c2db
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

add web API implementation for OSGi environment, based on Pax Web

usage of this in AAA can be seen in the next commit ("chained")

Change-Id: If298047e2b295ca88d0494dc9733e1d91ee44a12
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

add new API for programmatic registration of web Servlet, Filter, etc.

implementation & usage of this can be seen in the next "chained" commits

The purpose of this API is to let projects with web components, such as
neutron, aaa or restconf, ditch their respective web.xml.  This will have
a number of advantages, some of which are documented in the JavaDoc of
the new WebServer interface and WebContext class.

see also discussion and interest from project neutron re. adoption on:
https://lists.opendaylight.org/pipermail/neutron-dev/2018-February/001587.html

This is the change originally raised in infrautils as
Ib2df87ca31a2bde547efbf73e0475a1cd64ea6ea, but now instead proposed
to aaa, as discussed during the Kernel Projects call on 2018/02/27.

Change-Id: Ib2fb02aa19e49aa482062f18ba84124a9a623364
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

AAA-168: Remove embedded h2 dependency

This was code inherited from a long time ago and punted around AAA
without any real cleanup.  I am not sure why the original authors
decided to embed the dependency rather than just import it, but this
causes several issues.  This patch removes the embedded h2 dependency
in favor of direct import.  While I recognize that other parts of
ImportPackage need to be cleaned, they will be done in a subsequent
patch since they are separate concepts than what is done here.
In other words, expect follow-ups to continuing cleaning.

aaa-cli-jar relied on aaa-shiro shading the com.h2database:h2 jar,
so I instead added it as a compile time dependency for the module
and extracted the appropriate files for the generated jar in the
maven-shade-plugin configuration.

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

AAA-167:  Refresh test cert

Test cert used in UT for AAA was expired causing failures.  For now,
a 10 year cert is added to unblock the release.  Later, the tests
will be refactored to generate the key on the fly.

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

AAA-166 fix (attempt)

Change-Id: Ib2217b035138d993a88b2bc279316fd14925ea73
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Bump versions by x.(y+1).z for next dev cycle

Change-Id: I135cb265c0a637da0b34907025ede4f686c9bcc3
Signed-off-by: Anil Belur <abelur@linuxfoundation.org>

Merge "AAA-163: implement isEnabled flag in user model"

AAA-163: implement isEnabled flag in user model

Make it possible to utilize "isEnabled" for users.  This has been
broken since inception since the folks submitted the code back in
2014, and was recently caught.  Basically, it was not implemented
period in the user lookup logic.

Additionally, changes were needed to the "User" API-- Jaxb
serialization was never working.  That is, the original authors
decided to use an "int" to represent enabled in H2, yet, tried to
map to a Boolean in the serdes process.  This means that calls to
User.isEnabled() always returned null, which is VERY bad practice.

The public methods were left in place, some wrappers were added, and
the internal implementation was changed to utilize an int for
User.enabled field, but still provide boolean the User.isEnabled()
predicate.

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

Merge "Eliminate infinite busy wait in KarafIniWebEnvironment"

Eliminate infinite busy wait in KarafIniWebEnvironment

We've seen SFT's hang on shut down due to the busy wait if the
blueprint container for aaa-shiro fails or is destroyed while
being created. The busy wait blocks startup of the web app and also
blocks on shut down causing the hang. To alleviate this, use a
Future to obtain the AAAShiroProvider. Also added a BundleActivator
to register a blueprint event listener that fails the Future
exceptionally when the blueprint container is destroyed.

Change-Id: I9198579aa026ab685af73ee624c353d7097ba86e
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>

AAA-165: Add explicit import apache.commons.text

- added a simple UT for domain delete, but even that UT
  would not fail for AAA-165

- moved from commons-lang3 to commons-text as I noticed
  StringEscapeUtils is deprecated in lang3 and text was
  what should be used

Change-Id: I0e8a5067666d062e2f119ddaa7511f0cc3a2dda1
Signed-off-by: Jamo Luhrsen <jluhrsen@redhat.com>

AAA-134:  Cannot update domain name

Modified the update SQL statement to include name.

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

Merge "AAA-147: Jolokia Credentials are backed by AAA"

Merge "Move AAAShiroProvider class in act into separate package"

AAA-147: Jolokia Credentials are backed by AAA

Inject an Authenticator implementation into the service registry
for use with odl-jolokia from controller.  Corresponding patch:

https://git.opendaylight.org/gerrit/68069

W/o this Authenticator installed, jolokia authentication fails.

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

Move AAAShiroProvider class in act into separate package

There are two classes named AAAShiroProvider both in package
org.opendaylight.aaa; one in artifact (and OSGi bundle) aaa-shiro and
another one in aaa-shiro-act.

As far as I understand the AAA code, this does not seem to be done
intentionally for any particular reason, probably just historical
copy/paste.

Under OSGi, this doesn't really cause any major issues (other than it
being "really not nice", possibly confusing, and theoretically causing
issues if one were to use package import), as the two AAAShiroProvider
are loaded into separate ClassLoaders in their respective bundles.

When attempting to use AAA in a (non-OSGi) flat classpath environment
however, then the present situation, leads to this error:

[main] ERROR org.apache.shiro.web.env.EnvironmentLoader - Shiro
environment initialization failed
java.lang.NoSuchMethodError:
org.opendaylight.aaa.AAAShiroProvider.getInstance()Lorg/opendaylight/aaa/AAAShiroProvider;
at org.opendaylight.aaa.shiro.web.env.KarafIniWebEnvironment.init(KarafIniWebEnvironment.java:67)

This change fixes above, for non-OSGi test envs, but seems a reasonable
clean up if for the standard deployment model of running ODL in OSGi.

Change-Id: Ia5d34e6f3ecf9d6539bdac9537b8628dcde59049
Signed-off-by: Michael Vorburger <vorburger@redhat.com>

Clean up artifacts

This removes a number of obsolete artifacts from aaa-artifacts, adds
some new ones, and adds a new module to verify that listed artifacts
are actually available.

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

AAA-158: remove exception output in HTTP response

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

Fix README refactor

During the refactor that eliminated "impl" from the packaging
structure, this README was not correctly updated.

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

AAA-160: Fix aaa-cli commands

Adding final to the command line variable make the variable
finally initalized and not re-assign value at runtime.

Access the datastore using AaaShiroProvider and delete the
old datastore service.

Change-Id: I8c2f7d39bd5e9be5349a2b922bb8afdbbb0b15a3
Signed-off-by: serngawy <m.elserngawy@gmail.com>

Merge "AAA-159: Switch to using gson for JSON serialization"

AAA-159: Switch to using gson for JSON serialization

Due to jackson incompatibility issues with other web env
elements (namely jax-rs), this change switches the serdes
for AAA endpoints to gson.  The motivation is two fold:

1) fix the immediate issue with the rest endpoints
2) align on a common JSON serdes framework in ODL

Since yangtools and others already utilize gson, and gson
seems to be a lot more friendly from a provider stand-
point, this change is the best solution to the given bug.

This patch does not completely remove jackson, since RESTCONF
depends on us bringing it in.  This will be another multi-
step process:

1) this patch
2) convert restconf to GSON or just add the jackson deps
   there (features/odl-aaa-shiro/pom.xml changes in last
   patch)
3) remove jackson dependencies from AAA

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

ODLPARENT-139: re-use caches when restarting

When the Shiro bundles restart, for whatever reason, they always
attempt to re-create their caches; that fails because the caches are
still present. This patch re-uses existing cache managers and caches
when possible.

This doesn’t entirely resolve the SSH issues reported in
ODLPARENT-139, but it helps the SSH connection survive longer (which
makes it easier to debug).

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

NETCONF-502:  Provide GsonProvider for JSON serdes

Just provide the GsonProvider for downstream consumption.
Transitioning AAA to this provider will happen in a later
patch in order to avoid breaking downstreams in the
meantime.

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