aaa.git
4 years agoAdd PasswordCredentialAuth 88/89388/3
Robert Varga [Sun, 26 Apr 2020 01:32:13 +0000 (03:32 +0200)]
Add PasswordCredentialAuth

CredentialAuth is a generic interface which needs to be specialized
to capture capture which credentials are being passed. Downstreams
(in netconf) have a FIXME related to this, as they have no way to
express the type-safe dependency.

Introduce PasswordCredentialAuth to which downstreams can safely
bind. Also add a method to allow run-time discovery of required
credetial type.

Change-Id: I844db3b460bf95110fb3adbb687ce25e996e3608
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoUpdate MRI projects for Aluminium 05/88905/16
Robert Varga [Mon, 6 Apr 2020 09:52:40 +0000 (11:52 +0200)]
Update MRI projects for Aluminium

This bumps versions to the following:
- odlparent-7.0.1
- yangtools-5.0.1
- mdsal-6.0.0
- controller-2.0.0

Change-Id: Id5ffcc42659550a2bc0fb6a8ca8a334ffa3d34a0
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoClean up lamdas and methods in PaxWebServer 81/89381/2
Robert Varga [Sat, 25 Apr 2020 20:06:30 +0000 (22:06 +0200)]
Clean up lamdas and methods in PaxWebServer

Change the code a bit to make it more clean with respect to
references.

Change-Id: Ib86dcb2b8ee209d346d96dadecd4106c4d01251c
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoRemove intermediate ServiceFactory 79/89379/1
Robert Varga [Sat, 25 Apr 2020 19:25:33 +0000 (21:25 +0200)]
Remove intermediate ServiceFactory

Since PaxWebServer implements nothing we can easily make it a
ServiceFactory, reducing one indirection and making things a bit
more clear.

Change-Id: I41385adec33fb4b0ea17f3f74140090d8642540a
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoUse type-safe registration 78/89378/1
Robert Varga [Sat, 25 Apr 2020 19:13:30 +0000 (21:13 +0200)]
Use type-safe registration

OSGi R5+ is using generics, which offer type safety during registration.
Also remove unneded use of an empty dictionary -- null works just as well.

Change-Id: I3b062add5f05611af614da32c09810604b9a86dc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMerge "AAA-194 Fix for Pattern Matching in Shiro"
Robert Varga [Fri, 17 Apr 2020 16:06:10 +0000 (16:06 +0000)]
Merge "AAA-194 Fix for Pattern Matching in Shiro"

4 years agoRemove jetty-servlet-tester references 93/88993/1
Robert Varga [Thu, 9 Apr 2020 10:44:29 +0000 (12:44 +0200)]
Remove jetty-servlet-tester references

These are ancient and we do not seem to need them anymore, remove
them.

Change-Id: I4e7c3315aa53526cc2ce630db9cbe9f657fd45be
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoAAA-194 Fix for Pattern Matching in Shiro 38/88538/2
Venkatrangan Govindarajan [Thu, 19 Mar 2020 18:46:44 +0000 (00:16 +0530)]
AAA-194 Fix for Pattern Matching in Shiro

The Path Matching is based on AntPathMatcher
(https://docs.spring.io/spring-framework/docs/current/javadoc-api
/org/springframework/util/AntPathMatcher.html).
This Patch wil ensure the default urls also match in Shiro.

Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Change-Id: Iaa7a18d9138dad756eda24a31190421b07f25594

4 years agoFix checkstyle 06/88906/2
Robert Varga [Mon, 6 Apr 2020 10:06:45 +0000 (12:06 +0200)]
Fix checkstyle

Upgraded checkstyle does not like the mess in these declarations, clean them up.

Change-Id: Id6b42dc273f25ad716f048003d64a21022b08871
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMigrate Assert.assertThat() 08/88908/1
Robert Varga [Mon, 6 Apr 2020 10:16:27 +0000 (12:16 +0200)]
Migrate Assert.assertThat()

Migrate to non-deprecated version of assertThat().

Change-Id: I13a0387c46e758c3ba66fc0d38513311379d68d3
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMigrate OSGi compendium references 04/88904/2
Robert Varga [Mon, 6 Apr 2020 09:59:11 +0000 (11:59 +0200)]
Migrate OSGi compendium references

We are using OSGi R6, make sure we are also referencing the correct
compendium artifact name.

Change-Id: Ie142c309aa6b128e6223bf62d46dd2ad9ea50d8d
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMerge "Use Magnesium controller"
Robert Varga [Tue, 31 Mar 2020 05:46:36 +0000 (05:46 +0000)]
Merge "Use Magnesium controller"

4 years agoMaster branch is now Aluminium 22/88722/1
Thanh Ha [Thu, 26 Mar 2020 18:53:50 +0000 (14:53 -0400)]
Master branch is now Aluminium

Signed-off-by: Thanh Ha <zxiiro@gmail.com>
Change-Id: I8e83c90ec9f84cf3b582658fee0061d2be6d60c5

4 years agoUse Magnesium controller 74/88674/1
Robert Varga [Wed, 25 Mar 2020 21:39:21 +0000 (22:39 +0100)]
Use Magnesium controller

As part of controller transition to being MRI, downgrade the version
used to Magnesium release. This will be bumped to Aluminium proper
during the MRI window.

Change-Id: Ifb020b6b4d2b133c51874babfe0f49865b6d260b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMigrate rtd docs to python3 80/88180/1
Anil Belur [Sun, 1 Mar 2020 03:26:36 +0000 (13:26 +1000)]
Migrate rtd docs to python3

RTD builds have been failing on python2 since the required version
sphinx is not found in lftools-conf. Moving to python3 resolves the
issue.

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

4 years agoCleanup KeyStoreConstant 13/88013/2
Robert Varga [Sun, 23 Feb 2020 08:49:08 +0000 (09:49 +0100)]
Cleanup KeyStoreConstant

Java 11 provides utility methods which makes it much easier to
interact with files. Use them instead of verbose code doing the
same thing.

Change-Id: I1cf5bb3e307bc31f2301b4537fc718d0c9d29d00
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoFix NPE when loading certificate 30/87530/6
gvrangan [Mon, 10 Feb 2020 10:55:44 +0000 (10:55 +0000)]
Fix NPE when loading certificate

This Patch fixes the NPE when using a certificate from a
non-default location.

JIRA: AAA-191
Change-Id: If12d49d45d2e914bdc901ee692c02790bf9b51b9
Signed-off-by: gvrangan <vgovindarajan@luminanetworks.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump versions by x.y.(z+1) 84/87984/1
jenkins-releng [Sat, 22 Feb 2020 04:30:15 +0000 (04:30 +0000)]
Bump versions by x.y.(z+1)

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

4 years agoAAA-180: Fix Dynamic authorization 33/87733/1
Venkatrangan Govindarajan [Thu, 13 Feb 2020 08:56:39 +0000 (14:26 +0530)]
AAA-180: Fix Dynamic authorization

Repeated trials indicate the initfilter from shiro is never invoked for
this filter causing the problem. This patch instead uses the other metod(ProcessPAthConfig)
for initialization.

Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Change-Id: I30195bf745ac30f0cfd59021cb0b2fe524bede02
(cherry picked from commit 5b35f181c0315f5bcb99f95bd9a2ab4ea92122ef)

4 years agoFix variable name s/newUser/new_user/ 72/87072/1
Jamo Luhrsen [Wed, 22 Jan 2020 00:09:32 +0000 (16:09 -0800)]
Fix variable name s/newUser/new_user/

this bug was introduced with a pep8 refactor done in
https://git.opendaylight.org/gerrit/c/aaa/+/86832

Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
Change-Id: I26a3c80df1e49d1514bcceb7d7a93eeae98b7e1c

4 years agoRemove infrautils-testutils dependency 56/87056/1
Robert Varga [Mon, 20 Jan 2020 18:46:37 +0000 (19:46 +0100)]
Remove infrautils-testutils dependency

We do not need this dependency, remove it.

Change-Id: I14d328d25b6a8c05c6a0580495a0264d1e76b261
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoSwitch to JUnit assertThrows() 54/87054/1
Robert Varga [Mon, 20 Jan 2020 16:47:00 +0000 (17:47 +0100)]
Switch to JUnit assertThrows()

We have it available, there is no point to open-code it anymore.

Change-Id: I54274591545852bb7b698f436f7cf5581aa5494b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9 95/86995/2
Robert Varga [Thu, 16 Jan 2020 21:09:23 +0000 (22:09 +0100)]
Bump to odlparent-6.0.4/yangtools-4.0.6/mdsal-5.0.9

Pick up latest fixes and upgrades.

Change-Id: I41f80e0822a3232c8a7b1a3077322b9727b2215f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoFix python pep8 warnings 32/86832/3
Jamo Luhrsen [Thu, 9 Jan 2020 20:08:57 +0000 (12:08 -0800)]
Fix python pep8 warnings

a few warnings were not fixed like line length over 79 and
using bare except. Those can be done later if desired.

Change-Id: Ib21f7fc803daa34f696ad3a2ad277ae8f6657b04
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
4 years agoCatch missing arguments in python3 31/86831/3
Jamo Luhrsen [Thu, 9 Jan 2020 19:50:35 +0000 (11:50 -0800)]
Catch missing arguments in python3

See inline code comments for more details

JIRA: AAA-193
Change-Id: Id4de48147aaef26a7bb03dd00cf24dbe0d77fb05
Signed-off-by: Jamo Luhrsen <jluhrsen@gmail.com>
4 years agoDrop dependencies on commons-text 26/86726/2
Robert Varga [Sun, 5 Jan 2020 18:34:38 +0000 (19:34 +0100)]
Drop dependencies on commons-text

We are only using commons-lang3, which is a transitive dependency
of commons-text. Remove the unintended dependency.

Change-Id: I3d8a42703996de800926cb20b17c1b65672e9324
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoUpdate immutables.org integration 02/86402/5
Robert Varga [Thu, 12 Dec 2019 08:00:29 +0000 (09:00 +0100)]
Update immutables.org integration

Use only annotations, the rest is already configured from odlparent.

Change-Id: I6935b867f8e412128beb9f37b4872a20cc279c61
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump odlparent/yangtools/mdsal to 6.0.3/4.0.3/5.0.6 66/86366/6
Robert Varga [Thu, 12 Dec 2019 07:00:29 +0000 (08:00 +0100)]
Bump odlparent/yangtools/mdsal to 6.0.3/4.0.3/5.0.6

Pick up the latest upgrades and fixes.

Change-Id: I5a7098202d353bc1f9de761db66ae84e42a4c851
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoRemove superfluous feature dependencies 01/86401/1
Robert Varga [Thu, 12 Dec 2019 08:04:36 +0000 (09:04 +0100)]
Remove superfluous feature dependencies

We are generating features automatically, do not explicitly
specify dependencies, as they are picked up as needed.

Change-Id: I14cd2079a22d264dfa55dcf254fba59dd005da7e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4 15/85615/1
Robert Varga [Wed, 6 Nov 2019 14:08:12 +0000 (15:08 +0100)]
Bump to odlparent-6.0.1/yangtools-4.0.2/mdsal-5.0.4

This picks up latest upgrades and fixes.

Change-Id: I8db358b0af81394a561071d98bffea168dda47cc
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoMerge "Do not use toString() with loggers"
Robert Varga [Wed, 30 Oct 2019 12:33:18 +0000 (12:33 +0000)]
Merge "Do not use toString() with loggers"

4 years agoMerge "Updated the user guide after testing"
Robert Varga [Wed, 30 Oct 2019 12:32:25 +0000 (12:32 +0000)]
Merge "Updated the user guide after testing"

4 years agoFix idmtool.py for handling errors 74/84674/8
Venkatrangan Govindarajan [Mon, 23 Sep 2019 16:42:50 +0000 (22:12 +0530)]
Fix idmtool.py for handling errors

a. Modify the exiting method for checking REST reponses
and report success/failure.
b. Handle intermediate GET failures

JIRA: AAA-114
JIRA: AAA-120
Change-Id: I981e46859a2313a0f782b012920b2ab824ffcf44
Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoDo not use toString() with loggers 05/85405/1
Robert Varga [Mon, 28 Oct 2019 12:39:17 +0000 (13:39 +0100)]
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>
4 years agoCleanup JettyWebServer suppressions 85/85385/1
Robert Varga [Fri, 25 Oct 2019 11:22:48 +0000 (13:22 +0200)]
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>
4 years agoAggregators should use odlparent-lite 72/85372/1
Robert Varga [Thu, 24 Oct 2019 11:02:50 +0000 (13:02 +0200)]
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>
4 years agoRemove comons-beanutils overrides 68/85368/1
Robert Varga [Thu, 24 Oct 2019 10:46:52 +0000 (12:46 +0200)]
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>
4 years agoRemove install/deploy plugin configuration 67/85367/1
Robert Varga [Thu, 24 Oct 2019 10:10:03 +0000 (12:10 +0200)]
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>
4 years agoMerge "Ditch dependency on infrautils.testutils"
Robert Varga [Thu, 24 Oct 2019 08:43:04 +0000 (08:43 +0000)]
Merge "Ditch dependency on infrautils.testutils"

4 years agoMerge "Fix Typo in RPC Name"
Robert Varga [Thu, 24 Oct 2019 08:41:36 +0000 (08:41 +0000)]
Merge "Fix Typo in RPC Name"

4 years agoDitch dependency on infrautils.testutils 85/84785/12
Robert Varga [Fri, 27 Sep 2019 13:34:35 +0000 (15:34 +0200)]
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>
4 years agoFixup aaa-cert-mdsal pyang warnings 38/85338/1
Robert Varga [Wed, 23 Oct 2019 08:42:41 +0000 (10:42 +0200)]
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>
4 years agoRemove unneeded checkstyle/spotbugs declaration 80/85280/1
Robert Varga [Mon, 21 Oct 2019 03:56:34 +0000 (05:56 +0200)]
Remove unneeded checkstyle/spotbugs declaration

odlparent is enabling these, remove superfluous configuration.

Change-Id: Ib3b54f3db0a52c52197e47a5adfe7a572b55f56f
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoFix Typo in RPC Name 62/84762/9
Venkatrangan Govindarajan [Fri, 27 Sep 2019 00:02:06 +0000 (05:32 +0530)]
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>
4 years agoRemove use of jdepend.maven.plugin.version 94/85094/1
Robert Varga [Sun, 13 Oct 2019 08:21:21 +0000 (10:21 +0200)]
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>
4 years agoBump mdsal to 5.0.3 62/85062/1
Robert Varga [Fri, 11 Oct 2019 14:10:03 +0000 (16:10 +0200)]
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>
4 years agoDisable slf4j SSL link 41/85041/1
Robert Varga [Thu, 10 Oct 2019 18:57:42 +0000 (20:57 +0200)]
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>
4 years agoBump mdsal to 5.0.2 58/83858/14
Robert Varga [Wed, 21 Aug 2019 10:56:39 +0000 (12:56 +0200)]
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>
4 years agoUpdated the user guide after testing 52/84752/5
Venkatrangan Govindarajan [Thu, 26 Sep 2019 17:39:28 +0000 (23:09 +0530)]
Updated the user guide after testing

Change-Id: Ida7ddee17c36290767257356ec568be968b0d350
Signed-off-by: Venkatrangan Govindarajan <vgovindarajan@luminanetworks.com>
4 years agoBump yangtools to 4.0.1 57/83857/10
Robert Varga [Wed, 21 Aug 2019 10:55:15 +0000 (12:55 +0200)]
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>
4 years agoReduce javadoc links 52/84852/1
Robert Varga [Tue, 1 Oct 2019 15:08:30 +0000 (17:08 +0200)]
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>
4 years agoBump odlparent to 6.0.0 56/83856/8
Robert Varga [Wed, 21 Aug 2019 10:56:06 +0000 (12:56 +0200)]
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>
4 years agoFixup web-api tests 83/84783/2
Robert Varga [Fri, 27 Sep 2019 13:07:30 +0000 (15:07 +0200)]
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>
4 years agoAdd modernizer suppressions 55/83855/4
Robert Varga [Wed, 21 Aug 2019 12:38:09 +0000 (14:38 +0200)]
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>
4 years agoBump mdsal to 4.0.6 19/84419/2
Robert Varga [Sat, 14 Sep 2019 01:36:25 +0000 (03:36 +0200)]
Bump mdsal to 4.0.6

This brings in the latest fixes.

Change-Id: Ib5b6a2cc0fa56b5b7d999d4ec6f74feedd3951ef
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump yangtools to 3.0.5 00/84400/2
Robert Varga [Fri, 13 Sep 2019 14:27:12 +0000 (16:27 +0200)]
Bump yangtools to 3.0.5

This picks up the latest improvements.

Change-Id: Iad3b646f63a274022f004fa9f3d0b2ebdf0cf9db
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump odlparent to 5.0.2 49/83849/4
Robert Varga [Wed, 21 Aug 2019 10:39:34 +0000 (12:39 +0200)]
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>
4 years agoMerge "Update docs header to Magnesium in the master"
Robert Varga [Sat, 7 Sep 2019 06:20:50 +0000 (06:20 +0000)]
Merge "Update docs header to Magnesium in the master"

4 years agoApply code modernizations 54/83854/1
Robert Varga [Wed, 21 Aug 2019 12:29:30 +0000 (14:29 +0200)]
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>
4 years agoMigrate Vector to ArrayList 53/83853/1
Robert Varga [Wed, 21 Aug 2019 12:10:20 +0000 (14:10 +0200)]
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>
4 years agoMigrate deprecated newInstance() 52/83852/1
Robert Varga [Wed, 21 Aug 2019 11:02:51 +0000 (13:02 +0200)]
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>
4 years agoFix checkstyle 51/83851/1
Robert Varga [Wed, 21 Aug 2019 12:15:35 +0000 (14:15 +0200)]
Fix checkstyle

This fixes up checkstyle warnings reported by updated version.

Change-Id: I91fd55a8817630c36e673adc3b8569a82aa315f6
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoRemove unneeded assert 50/83850/1
Robert Varga [Wed, 21 Aug 2019 11:02:26 +0000 (13:02 +0200)]
Remove unneeded assert

This assertion is always true, remove it.

Change-Id: I8e24a6bfb3c662dfc4e96df77d1df79d81a2380b
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoUpdate docs header to Magnesium in the master 05/83805/1
Anushka Bhandari [Tue, 20 Aug 2019 16:52:27 +0000 (22:22 +0530)]
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

4 years agoBump mdsal to 4.0.4 28/83728/1
Robert Varga [Thu, 15 Aug 2019 08:28:19 +0000 (10:28 +0200)]
Bump mdsal to 4.0.4

This brings in latest fixes.

Change-Id: I59f89ef4fe76ef16fcdc9b59f8d9b50aa0849847
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoBump versions by x.y.(z+1) 41/83541/1
jenkins-releng [Mon, 12 Aug 2019 16:14:11 +0000 (16:14 +0000)]
Bump versions by x.y.(z+1)

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

4 years agoupdate dev-guide.rst 92/83492/3
nullop [Tue, 6 Aug 2019 16:12:47 +0000 (12:12 -0400)]
update dev-guide.rst

update the instructions for servlet support

Issue-ID: AAA-185
Change-Id: Ibc8537f33893d82f3bb5fc924a5d714c7d428267
Signed-off-by: nullop <ks6305@att.com>
4 years agoBump mdsal to 4.0.3 98/83098/5
Robert Varga [Thu, 18 Jul 2019 10:32:21 +0000 (12:32 +0200)]
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>
4 years agoBump yangtools to 3.0.4 97/83097/4
Robert Varga [Thu, 18 Jul 2019 10:31:44 +0000 (12:31 +0200)]
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>
4 years agoBump odlparent to 5.0.1 96/83096/1
Robert Varga [Thu, 18 Jul 2019 10:31:21 +0000 (12:31 +0200)]
Bump odlparent to 5.0.1

This picks up the latest upgrades.

Change-Id: If6006a94197839a9718cd5eae16b0a025913af93
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
4 years agoFix JDK11/docs compilation 41/82941/1
Robert Varga [Mon, 8 Jul 2019 08:17:01 +0000 (10:17 +0200)]
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>
4 years agoAdd aaa-docs 03/82903/1
Robert Varga [Tue, 28 May 2019 09:15:36 +0000 (11:15 +0200)]
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>
5 years agoInitialize MDSALDynamicAuthorizationFilter lazily 69/82569/2
Robert Varga [Wed, 19 Jun 2019 13:13:28 +0000 (15:13 +0200)]
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>
5 years agoBump mdsal to 4.0.2 68/82268/3
Robert Varga [Tue, 28 May 2019 07:19:05 +0000 (09:19 +0200)]
Bump mdsal to 4.0.2

This picks up the latest improvements.

Change-Id: Ic51617c1ef8574c949995d81daad4e6b2ae3db07
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoBump yangtools to 3.0.1 67/82267/2
Robert Varga [Tue, 28 May 2019 07:18:49 +0000 (09:18 +0200)]
Bump yangtools to 3.0.1

This picks up the latest fixes.

Change-Id: I5aa12ab70664b573710a7648df2bc61b6fbeb013
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
5 years agoBump mdsal to 4.0.1 24/81824/3
Robert Varga [Tue, 30 Apr 2019 11:48:24 +0000 (13:48 +0200)]
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>
5 years agoAdd web-jetty-impl to artifacts 24/82024/1
Robert Varga [Tue, 14 May 2019 10:31:54 +0000 (12:31 +0200)]
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>
5 years agoBump mdsal to 4.0.0 68/81268/8
Robert Varga [Thu, 28 Mar 2019 18:51:06 +0000 (19:51 +0100)]
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>
5 years agoBump yangtools to 3.0.0 67/81267/7
Robert Varga [Thu, 28 Mar 2019 18:49:55 +0000 (19:49 +0100)]
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>
5 years agoBump odlparent to 5.0.0 66/81266/5
Robert Varga [Thu, 28 Mar 2019 18:47:54 +0000 (19:47 +0100)]
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>
5 years agoDo not unregister oauth2 endpoint 88/81488/1
Robert Varga [Tue, 9 Apr 2019 13:13:28 +0000 (15:13 +0200)]
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>
5 years agoFix warnings in aaa-filterchain 70/81270/1
Robert Varga [Thu, 28 Mar 2019 19:18:55 +0000 (20:18 +0100)]
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>
5 years agoMerge "Remove dependency on features-test"
Tom Pantelis [Fri, 22 Mar 2019 21:52:39 +0000 (21:52 +0000)]
Merge "Remove dependency on features-test"

5 years agoRemove dependency on features-test 13/81113/1
Robert Varga [Fri, 22 Mar 2019 12:52:28 +0000 (13:52 +0100)]
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>
5 years agoAAA-174 : Don't reuse JDBC connection 08/81108/2
Richard Kosegi [Fri, 22 Mar 2019 08:18:03 +0000 (09:18 +0100)]
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>
5 years agoEliminate AAA OAuth2 Provider 22/72022/14
Ryan Goulding [Tue, 15 May 2018 16:24:38 +0000 (12:24 -0400)]
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>
5 years agoRemove javax.annotation-api version overrides 48/80948/1
Robert Varga [Mon, 18 Mar 2019 12:29:13 +0000 (13:29 +0100)]
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>
5 years agoEliminate powermock from aaa-encrypt-service 27/80727/1
Robert Varga [Thu, 7 Mar 2019 16:11:12 +0000 (17:11 +0100)]
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>
5 years agoMigrate to ArgumentMatchers 20/80720/2
Robert Varga [Thu, 7 Mar 2019 14:15:14 +0000 (15:15 +0100)]
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>
5 years agoRemove powermock from KeyStoresDataUtilsTest 19/80719/2
Robert Varga [Thu, 7 Mar 2019 14:12:58 +0000 (15:12 +0100)]
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>
5 years agoRemove powermock from AaaCertProviderTest 18/80718/2
Robert Varga [Thu, 7 Mar 2019 13:59:54 +0000 (14:59 +0100)]
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>
5 years agomake JettyWebServer accept WebContext path without slash (and test it) 06/80206/1
Michael Vorburger [Thu, 7 Feb 2019 13:23:30 +0000 (14:23 +0100)]
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>
5 years agoBump versions by x.y.(z+1) 81/80081/1
jenkins-releng [Fri, 1 Feb 2019 13:06:28 +0000 (13:06 +0000)]
Bump versions by x.y.(z+1)

Change-Id: Ie79426eab5edd155329862d04fe9ce44c7d0c656
Signed-off-by: jenkins-releng <jenkins-releng@opendaylight.org>
5 years agoFix idmtool print error in case of using python3 39/79939/1
Yi Yang [Mon, 28 Jan 2019 02:50:35 +0000 (21:50 -0500)]
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>
5 years agoMove BP xml files to standard OSGI-INF/blueprint 57/79857/1
Tom Pantelis [Wed, 23 Jan 2019 17:02:01 +0000 (12:02 -0500)]
Move BP xml files to standard OSGI-INF/blueprint

Change-Id: I79b9abc3e972fdea31eb5df4e53b05bc3565d35e
Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
5 years agoBump powermock to version 2.0.0 29/79829/1
Robert Varga [Tue, 22 Jan 2019 22:52:44 +0000 (23:52 +0100)]
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>
5 years agoBump to mdsal 3.0.6 05/79805/1
Stephen Kitt [Tue, 22 Jan 2019 08:49:41 +0000 (09:49 +0100)]
Bump to mdsal 3.0.6

Change-Id: I6432535435acbd096791d324cfa00903c986a469
Signed-off-by: Stephen Kitt <skitt@redhat.com>
5 years agoBump mdsal to 3.0.5 45/79745/4
Robert Varga [Sat, 19 Jan 2019 20:37:02 +0000 (21:37 +0100)]
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>
5 years agoBump yangtools to 2.1.8 44/79744/2
Robert Varga [Sat, 19 Jan 2019 20:37:37 +0000 (21:37 +0100)]
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>
5 years agoBump to odlparent 4.0.9 61/79661/1
Stephen Kitt [Fri, 18 Jan 2019 15:31:56 +0000 (16:31 +0100)]
Bump to odlparent 4.0.9

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