Do not use toString() with loggers

Formatting statements which are arguments to debug() is wasteful,
as logging implementation will do the right thing if/when needed.

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

Cleanup JettyWebServer suppressions

This cleans up exception handling, removing unneeded supressions
and lowering the scope of the checkstyle:IllegalCatch.

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

Aggregators should use odlparent-lite

These are mistakenly using full odlparent, which is not what
we want. Additionally aaa-password-service lists dependencies,
which is also wrong.

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

Remove comons-beanutils overrides

odlparent is propagating the proper definition, do not override
it here.

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

Remove install/deploy plugin configuration

Control install/deploy through a simple property rather than
spelling out the complete configuration for install/deploy plugins.

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

Merge "Ditch dependency on infrautils.testutils"

Merge "Fix Typo in RPC Name"

Ditch dependency on infrautils.testutils

We are depending on testutils only for WebClient. Java 11 provides
a simple HttpClient, which is way more powerful, use that rather
than a random home-grown thing.

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

Fixup aaa-cert-mdsal pyang warnings

This is just a matter of reordering statements, fix that up.

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

Remove unneeded checkstyle/spotbugs declaration

odlparent is enabling these, remove superfluous configuration.

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

Fix Typo in RPC Name

This is a simple typo, make sure we have a sane interface.

JIRA: AAA-186
Change-Id: I1ecceff4c860792664b6dbd17c8309b8bb43dbaf
Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove use of jdepend.maven.plugin.version

jdepend-maven-plugin is properly declared in odlparent, remove
the property reference.

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

Bump mdsal to 5.0.3

This bumps mdsal to 5.0.3 to workaround issues with Blueprint
injection.

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

Disable slf4j SSL link

slf4j does not provide apidocs over TLS (WTF?!?!?!), link HTTP
site instead.

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

Bump mdsal to 5.0.2

This bumps mdsal to 5.0.2 as part of Magnesium MRI.

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

Bump yangtools to 4.0.1

This bumps yangtools to 4.0.1 as part of Magnesium MRI.

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

Bump odlparent to 6.0.0

This bumps odlparent to 6.0.0 as part of Magnesium MRI.

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

Reduce javadoc links

There is a bunch of links we do not use -- just remove them.

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

Fixup web-api tests

This removes @Ignore from the two out of three tests, and fixes them
up to perform correct assertion. Also ditch dependency on infrautils
while we're at it -- while this has *some* duplication, we will ditch
that once we have JUnit 4.13 anyway.

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

Add modernizer suppressions

We are dealing with Dictionaries, hence we need to suppress
modernization.

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

Bump mdsal to 4.0.6

This brings in the latest fixes.

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

Bump yangtools to 3.0.5

This picks up the latest improvements.

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

Bump odlparent to 5.0.2

This bumps odlparent to pick up latest references.

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

Merge "Update docs header to Magnesium in the master"

Apply code modernizations

This migrates off of methods pointed out by modernizer plugin.

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

Migrate Vector to ArrayList

As pointed out by modernizer, we should be using ArrayList instead
of Vector.

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

Migrate deprecated newInstance()

Class.newInstance() is deprecated since Java 9, remove its use.

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

Fix checkstyle

This fixes up checkstyle warnings reported by updated version.

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

Remove unneeded assert

This assertion is always true, remove it.

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

Update docs header to Magnesium in the master

Should be Magnesium on master and Sodium on stable/sodium.

Signed-off-by: Anushka Bhandari <anushka16134@iiitd.ac.in>
Change-Id: I3e4db5d8431ebac5b55cdb00ccd6f3de9831f440

Bump mdsal to 4.0.4

This brings in latest fixes.

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

Bump versions by x.y.(z+1)

Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
Change-Id: I10ceb0ab12c777b0cb034a4e28054bb441973428

update dev-guide.rst

update the instructions for servlet support

Issue-ID: AAA-185
Change-Id: Ibc8537f33893d82f3bb5fc924a5d714c7d428267
Signed-off-by: nullop <ks6305@att.com>

Bump mdsal to 4.0.3

This bumps mdsal to 4.0.3, bringing in latest fixes and improvements.

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

Bump yangtools to 3.0.4

This picks up latest fixes and improvements.

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

Bump odlparent to 5.0.1

This picks up the latest upgrades.

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

Fix JDK11/docs compilation

JDK11 is more picky about errors, this fixes them up.

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

Add aaa-docs

This adds documentation aggregator, which produces a aaa-docs.jar,
containing all AAA documentation we have in-tree.

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

Initialize MDSALDynamicAuthorizationFilter lazily

Filters should not initialize in their constructor, but rather
when instructed to do so by the container. This patch moves
initialization to onFilterConfigSet(), which is called from
the init() method.

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

Bump mdsal to 4.0.2

This picks up the latest improvements.

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

Bump yangtools to 3.0.1

This picks up the latest fixes.

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

Bump mdsal to 4.0.1

This bumps mdsal to 4.0.1.

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

Add web-jetty-impl to artifacts

Failing to publish the proper version is a mistake forcing
downstreams to specify multiple versions. Fix that.

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

Bump mdsal to 4.0.0

This bumps mdsal to 4.0.0.

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

Bump yangtools to 3.0.0

This bumps yangtools to 3.0.0, with no additional changes required.

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

Bump odlparent to 5.0.0

This bumps odlparent dependencies to 5.0.0, removing references
to javax.annotation.concurrent, as they are not that useful.

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

Do not unregister oauth2 endpoint

Since we are not registering the endpoint, we should not be
unregistering it.

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

Fix warnings in aaa-filterchain

These are simple uses of deprecated methods, missing serialVersion
and unneded suppression.

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

Merge "Remove dependency on features-test"

Remove dependency on features-test

This dependency should not be used anywhere, remove it.

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

AAA-174 : Don't reuse JDBC connection

Multiple threads can obtain same instance of Connection
which is not thread safe. After first thread call #close()
other threads can't use connection further.

Proper solution would be connection pool, but as H2 store
is local, obtaining new connection is cheap.
Local testing doesn't yield measurable performence penality.

Change-Id: Ie326e39da8cb01283cc516ef99363c0f4236698d
Signed-off-by: Richard Kosegi <richard.kosegi@gmail.com>

Eliminate AAA OAuth2 Provider

For the foreseeable past, OpenDaylight AAA development team has
agreed to avoid becoming an IdP.  The simple fact is that there
hundreds (if not more) of IdP solutions that are very secure and
have been proved with time.  The cost of maintaining an IdP is
extremely high, as the code is not always simple, and requires
highly skilled, security minded developers to maintain.

However, one remenant of IdP currently exists as a part of ODL
AAA, namely the OAuth2 Provider Implementation.  The OAuth2
Provider implementation allows users to derive scoped access
tokens in order to access the controller without using plain
credentials.  Part of OAuth2 inherently also provides easy
revocation of tokens.

However, this begs the question, are we an IdP at this point?
The preferred security model would be to delegate to an external,
third-party OAuth2 Provider, since many exist and are readily
available for federation.  This would limit the scope to
client-only for AAA.  This is preferred since AAA should not
be responsible for generation and revocation of tokens.  The
existing implementation utilized Apache OLTU, which has now
been retired to the Attic.

This patch favors removing unmaintained, possibly vulnerable
security code over maintaining OAuth2 backwards compatibility.
The functionality is not being replaced for two reasons:

1) we strive not to act as and IdP
2) most other OAuth2 provider solutions are spring-security based,
and are fairly difficult to use from non-spring contexts.

JIRA: AAA-173
Change-Id: I090014771b2f345cedc0330738b15b18684e1fcf
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>

Remove javax.annotation-api version overrides

odlparent-4.0.9 is integrated, hence the override is not needed
anymore.

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

Eliminate powermock from aaa-encrypt-service

The single test here is easily reformulated in terms of mockito,
hence remove static mocking and powermock.

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

Migrate to ArgumentMatchers

This fixes warnings in aaa-cert around use of deprecated mockito
methods.

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

Remove powermock from KeyStoresDataUtilsTest

This test uses static method mocking, but that can be easily
fixed by injecting proper DataBroker behavior.

Remove static mocking, eliminating the need for powermockito
in aaa-cert.

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

Remove powermock from AaaCertProviderTest

This test does not require powermock, use plain mockito instead.

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

make JettyWebServer accept WebContext path without slash (and test it)

because that is, apparently, what the "real" (Jetty through Pax Web)
implementation in the Karaf run-time does.  This has come up in the
review of change I614a48bcc425db8faa1a79ebd6236236bcd09e73, where I
added a slash to a context to get a test to pass - with this, that is no
longer needed.

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

Bump versions by x.y.(z+1)

Change-Id: Ie79426eab5edd155329862d04fe9ce44c7d0c656
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>

Fix idmtool print error in case of using python3

In Ubuntu 16.04, python3 is default python version, but idmtool
is still using non-strict print syntax, so it won't work if we
use python3, here is error information.

vagrant@odl:~$ python3 ./idmtool.py.orig
File "./idmtool.py.orig", line 143
print ''
^
SyntaxError: Missing parentheses in call to 'print'

In python3, print must use parentheses to embrace all the arguments.

Change-Id: Icec4c9c0225cae4c62e2189af9702789906d5e4f
JIRA: AAA-183
Signed-off-by: Yi Yang <yangyi01@inspur.com>

Move BP xml files to standard OSGI-INF/blueprint

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

Bump powermock to version 2.0.0

This bumps from RC.4 to proper release.

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

Bump to mdsal 3.0.6

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

Bump mdsal to 3.0.5

This aligns with odlparent-4.0.9 and yangtools-2.1.8.

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

Bump yangtools to 2.1.8

This align with odlparent-4.0.9.

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

Bump to odlparent 4.0.9

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

Merge "introduce aaa.web.testutils with WebTestModule & TestWebClient"

introduce aaa.web.testutils with WebTestModule & TestWebClient

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

Bump mdsal to 3.0.4

This realigns odlparent/yangtools/mdsal and brings in the latest
improvements.

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

Bump yangtools to 2.1.7

This realigns odlparent/yangtools and brings in a few minor
improvements.

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

Bump to odlparent 4.0.8

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

Drop javax.annotation-api scope

scope=provided does not seem to be necessary, remove it.

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

Add javax.annotation dependencies

Java 11 does not ship with javax.annotation package, hence
we need to declare our dependency on them.

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

Fix compilation with JDK11

PowerMock-1.7.4 just does not work with JDK11, update it to version
2.0.0.RC-4, which mostly works, and add javax.xml exclusions to
work around the remaining issue.

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

Use scope=provided for karaf console

We expect karaf shell to be provided by the base distribution,
hence we should not use scope=compile here.

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

Bump odlparent to 4.0.7

This fixes SFT compatibility with JDK11.

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

Switch to using spotbugs

findbugs throws ugly warnings and generally does not work with
JDK9+. Switch to spotbugs, which works fine.

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

Lower aaa-cli dependency on shell

karaf.shell.core is providing all the interfaces required for
this artifact, make sure we depend only on that.

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

Eliminate unneeded jaxb-api version/exclusion

odlparent is declaring the correct dependency, reuse it from there.

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

Use odl-jersey-2

odlparent is packaging a global version of Jersey, make sure we
use that feature instead of rolling our own.

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

Trim odl-aaa-api bundles

odlparent bundles servlet and JAX-RS APIs, use those features
instead of packaging artifacts ourselves.

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

Remove unneeded sshd-core dependency

This dependency was missed in
I6d96764f36c8d60944c68361e63d613120098946, remove it now as
it's completely defunct.

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

Use odl-apache-commons-codec

odlparent is packaging commons-codec in a feature, use that rather
than packaging it ourselves.

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

Remove unneeded commons-codec dependency

This dependency is not used in the feature, drop it.

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

Bump mdsal to 3.0.3

This brings in latest fixes and alignes with odlparent-4.0.5.

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

Bump to yangtools-2.1.6

This brings in latest fixes and odlparent-4.0.5 alignment.

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

Bump to odlparent 4.0.5

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

Do not override odlparent versions

odlparent is specifying 2.25.1 in dependencyManagement, there is
no need to override.

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

Require jaxb-api

Java 9+ does not contain jaxb-api classes, hence we need to add
an explicit dependency. Since karaf is providing it in runtime,
we also need corresponding feature exclusions -- which will be
rendered unnecessary with odlparent-4.0.3.

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

Update mockito references

This updates use of Mockito method to their non-deprecated
equivalents.

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

Merge "Bump to yangtools-2.1.4 and mdsal-3.0.2"

Simplify install/deploy skip

odlparent-lite handles magic properties, without the need for
plugin boiler-plate.

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

Add aggragate coverage support

This adds the build-system-side of aggragating jacoco reports
for sonar.

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

Bump to yangtools-2.1.4 and mdsal-3.0.2

This bumps the dependencies to 2.1.4 and 3.0.2, respectively.

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

add support for automatically choosing free port to JettyWebServer

This is useful for tests.  It could allow parallel running of web tests.

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

Migrate aaa-cli to JDT annotations

This eliminates a reference to javax.annotation from aaa-cli,
favoring JDT annotations instead.

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

Migrate aaa-filterchain to JDT annotations

Remove more dependencies on javax.annotation, favoring JDT
instead.

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

Migrate aaa-cert to JDT annotations

Remove use of JSR305 annotations in favor of JDT ones.

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

Remove unneeded javax.annotations from aaa-shiro

These are used only internally, remove them to lower dependencies
on javax.annotation.

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

Migrate aaa-shiro to MD-SAL APIs

This patch performs the trivial migration to MD-SAL APIs.

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