Update policy cookbook documentation

Change-Id: I86f99dfec5d8b402505dd5922c985e6b9e93ceb9
Issue-ID: POLICY-1167
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>
(cherry picked from commit 5a91dc31a529ff9d795e83650d400a5a35bf8755)

Upgrade to policy/common 1.3.2

I will change the drools applications when it is released.

Issue-ID: POLICY-1245
Change-Id: I06e9ce9dc7c17181b05dd583020c91dda6ca7aaf
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Upgrade xacml 1.0.2" into casablanca

Removed Objective From keyword check in parser

Modified the Model parser code to allow "objective" as a field name in
yaml.

Change-Id: Ia3667ebc84a0bbdbbb3723bdcf0237a8f35313ee
Issue-ID: POLICY-1244
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Upgrade xacml 1.0.2

This version of xacml only has license changes to the java
source files. This will clear ONAP license issues.

Issue-ID: POLICY-1228
Change-Id: I7496fc54e44a51f89e438adaa1ab70a1c23aa06d
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
(cherry picked from commit 510c439f70556718f9fceeb6bad1a771ccbb410c)

Modified to make AAF configurable in policy/engine

Added property to enable or disable AAF
Added logic to the code to check the property and use old authentication
logic to authenticate the user rather than AAF if disabled.

Fixed license issue
Modified cadi_truststore_password value to use proper variable
Added comment to local development enviroment properties file
Modified test resources to fix Junit failures

Change-Id: I962895154716092c9cc0bc6c48f6419f3b5900fe
Issue-ID: POLICY-1217
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Merge "Support R3 model for OOF Policy Optimization" into casablanca

Support R3 model for OOF Policy Optimization

I added an INSERT to the sql upgrade file for model 1.0
and delete for downgrade

Change-Id: I56a0045a75fef2c2cafe6c32ab87f2c9e1c0aa5e
Issue-ID: POLICY-1226
Signed-off-by: liboNet <libo.zhu@intel.com>
(cherry picked from commit 52f2c099fe6123381a9f4f528ff5acacb37b3ecd)

Remove openecomp repo from engine pom

Change-Id: I5651c65d06ceacc663b985315755d6cb93cd6666
Issue-ID: POLICY-1231
Signed-off-by: Jim Hahn <jrh3@att.com>
(cherry picked from commit 205eaacc4db3e853520cc49e0f62dba407f9a2ae)

Step version for Casablanca support

Issue-ID: POLICY-1233
Change-Id: I1d388e1f4a872cf4e3013f8968e9026460e847f3
Signed-off-by: liamfallon <liam.fallon@ericsson.com>

Fix random failures in ManualClientEndTest

Don't know if this is the cause of the intermittent failures, but it
could be, and it is certainly a bug: the classes manipulate data, that
isn't marked volatile, from more than one thread at a time.
Updated licenses.

Change-Id: Ifca84cf8f915ec703a9ce8980db2475dacf97980
Issue-ID: POLICY-1092
Signed-off-by: Jim Hahn <jrh3@att.com>

API Doc: Add examples

API Doc: Add examples
-------------------------------------------------------------

Change-Id: Ia219b06cdb05ef67c40c8920ea05087a0bf249d8
Issue-ID: POLICY-1219
Signed-off-by: Saryu Shah <ss3917@att.com>

Upgraded Tomcat version to 8.5.34

Upgraded Tomcat to resolve security vulnerability
http://mail-archives.us.apache.org/mod_mbox/www-announce/201810.mbox/<4cf697b0-db03-9eab-f2aa-54c2026d0e88@apache.org>

Issue-ID: POLICY-1212
Change-Id: I05f09e3fb862ad485dee519a0014e51a0c8d9ac3
Signed-off-by: mmis <michael.morris@ericsson.com>

Fix refering to snapshot in policy/engine

Issue-ID: POLICY-1213
Change-Id: I6666585a176b9e981976777018e4b7616e47ffc7
Signed-off-by: ramverma <ram.krishna.verma@ericsson.com>

Remove commons-upload

Still a couple more places to remove it due to security
violations. Unused dependency.

Issue-ID: POLICY-1214
Change-Id: Ie9a4755fa912a863be58f6ddec104702e68287d4
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Resolve security issues with postgresql

Removed the unused dependency

Change-Id: Iad55fc46a2912a297dd2ae0300fe99489c07d123
Issue-ID: POLICY-1215
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Made change to validation to reduce complexity

I had this change before Jorge merged the my previous commit for fixing
optimization policy validation and was planning to amend but Jorge was
quicker on the draw :)

Thanks Pam for pointing this out.. a simple way to reduce complexity
that I will use going forward.

Change-Id: I3ab8ae36591c7c69629f36d84261f95c559d0f42
Issue-ID: POLICY-1205
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Merge "Remove unused commons-fileupload"

Remove unused commons-fileupload

This flags a security vulnerability. But it is unused so
we can remove it.

Issue-ID: POLICY-1212
Change-Id: I8343d56324cc94d7a163f26d0af322d232c5c02e
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Fixed Optimization Policy validation

Modified the code to validate keys with empty array.

Change-Id: I8046d051f5bf7407a6e890f1ba3eca6baf6ac9eb
Issue-ID: POLICY-1205
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Update Casablance Release Notes

Update Casablance Release Notes
-------------------------------------------------------------

Issue-ID: POLICY-1162

Change-Id: I816745a20226278cef17dbe02591f7b001efb3a0
Signed-off-by: Saryu Shah <ss3917@att.com>

Updated policy documentation - vFW

Updated policy documentation - vFW
-------------------------------------------------------------

Change-Id: I65a1fa707f6443d3c947b80c2328ba43c0756aa2
Issue-ID: POLICY-1169
Signed-off-by: Saryu Shah <ss3917@att.com>

tutorial_cl, vCPE documentation

tutorial_cl, vCPE documentation
-------------------------------------------------------------

Change-Id: Ibed905fd8f8a27af5e7bae2024b15f5c02fff390
Issue-ID: POLICY-1169
Signed-off-by: Saryu Shah <ss3917@att.com>

Merge "Documentation: Eclipse, vCPE, VoLTE, ModTemplate"

Merge "Updated vDNS documentation"

Documentation: Eclipse, vCPE, VoLTE, ModTemplate

Documentation: Eclipse, vCPE, VoLTE, ModTemplate
-------------------------------------------------------------

Issue-ID: POLICY-1169

Change-Id: Id501913db5be23f97e5ea5a635e27511546a042a
Signed-off-by: Saryu Shah <ss3917@att.com>

Fix exceptions related to invalid db schema

Modified db upgrade script to alter systemlogdb table to allow null values for "remote" column.

Change-Id: Ib1bba59a38084bef8f89245d3725d18a02b303dc
Issue-ID: POLICY-1204
Signed-off-by: Michael Mokry <michael.mokry@att.com>

XACML PDP Performance Test

Added TestSuite module and performance test plan and stability test plan
that I received from Michael Morris

Added License headers to files per Jim's comment

Change-Id: I9d7a2740218a946ea031fdeae57dcb58d4ddd886
Issue-ID: POLICY-887
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Updated vDNS documentation

Updated vDNS documentation
-------------------------------------------------------------

Change-Id: I763996f48ae3665dddd8836994f77bed283806ee
Issue-ID: POLICY-1169
Signed-off-by: Saryu Shah <ss3917@att.com>

Merge "Policy Doc: Removed Beijing Controller"

Merge "Updated the keystore to include pdp wildcard."

Updated the keystore to include pdp wildcard.

This allows to talk directly to OOM PDP-X pods
without going through the service.

Change-Id: I09bd5253a79142f6566806b97be4bb6c223826f4
Issue-ID: POLICY-1198
Signed-off-by: Jorge Hernandez <jorge.hernandez-herrero@att.com>

Policy Doc: Removed Beijing Controller

Removed "Installation of Beijing Controller and Policies" from index
-------------------------------------------------------------

Issue-ID: POLICY-1196  (remove Beijing controller doc)

Change-Id: I7531e3a9074792542799fdc92179cca41592875f
Signed-off-by: Saryu Shah <ss3917@att.com>

Updated rst doc for clsimulation.rst

A short description eg. Updated policy documentation
-------------------------------------------------------------

Change-Id: Ibc2426f9b6befc2c750290f2685427a47318e012
Issue-ID: POLICY-1169
Signed-off-by: Saryu Shah <ss3917@att.com>

Regenerate policy keystore

The keystore has been regenerated using the latest tooling
from AAF.

Previous versions seem to add a bogus " fqdn" which makes
the particular policy.api.simpledemo.onap.org invalid.
The latest version does not append it.

Change-Id: I96e4d272e8f0ab6eb84a12384645bf9a49dbbd09
Issue-ID: POLICY-1193
Signed-off-by: Jorge Hernandez <jh1730@att.com>

Modify portal properties in PE

Modified the portal.properties with new role_access_centralized
property.

Modified the conf file with updated values for the ONAP redirect and
REST URL properties

Made changes for comments

Change-Id: Id2624cbb440ea5830520c5e82436d112989b62c4
Issue-ID: POLICY-1192
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Update local dependency json

For local testing these need to be added.

Issue-ID: POLICY-1190
Change-Id: I2827aebe9faf04ff38e0781376b4553d34c88930
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Initial Draft of Release Notes

Initial Draft of Release Notes
-------------------------------------------------------------

Change-Id: I76d9c4cdd4cef570a4e4bf6fc149cf28dd447297
Issue-ID: POLICY-1162
Signed-off-by: Saryu Shah <ss3917@att.com>

Fix sonar bug

Somehow this creeped into sonar. Not sure why it wasn't
flagged before.

Fixed tabs and indentation.

Removed unnecessary close() statement for a try-with-
resources statement.

Issue-ID: POLICY-1186
Change-Id: I5499279fe7adf642a7f7dc8a53c1a27c24a0dcb3
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Add SDNC actor to dependency

So BRMS gateway can add this dependency during runtime.

Issue-ID: POLICY-1183
Change-Id: I82127fca5e0d644463e82d543b6331dde8c32042
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Fixed issue with springframework dependencies

Added exclusions to epsdk dependencies and added dependencies for
springframework jars that were needed to the ONAP-SDK-APP

Change-Id: I037ffa76d3d32b2db5ea2c439f40e8a86e1665f5
Issue-ID: POLICY-1154
Signed-off-by: Michael Mokry <michael.mokry@att.com>

Recreate latest keystore.

Some changes have been made since the first set of
stores were generated from the cert manager tooling
in the windriver test instance used for this purpose.

Change-Id: I976333df7e020bda76c5cbdc9c0be752e5a88778
Issue-ID: POLICY-1156
Signed-off-by: Jorge Hernandez <jh1730@att.com>

Add Sdnr Actor code

These need to be defined for BRMS Gateway to re-created the
artifacts.

Issue-ID: POLICY-1155
Change-Id: I424be9976fa129a42e0020870af01ba30e0c0d47
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

CADI AAF changes for policy/engine

Added cadi properties and modified policy aaf client code.

Fixed issue with namespace and modified code to reverse it for
structuring the username sent in AAF API call

Added properties for keystore and keystore password after getting error
response from AAF when setting up the connection to AAF

Missed a fix for one of Jorge's comments in last patch, here it is.

Change-Id: Ic164ade8aa34da95a560c1592656e0caf990a595
Issue-ID: POLICY-913
Signed-off-by: Michael Mokry <mm117s@att.com>

BRMSGW HTTPS support for DMaaP/UEB interfaces

Modified BrmsPush UEB interface implementation to use HTTPS

Change-Id: Id8ae68f23ee371142da3f9d8aa0a2b2e7988ff36
Issue-ID: POLICY-942
Signed-off-by: Michael Mokry <mm117s@att.com>

Upgrade javax.mail to 1.6.2

The former version was frozen in github. It looks like they
moved to another website and the changes were transparent codewise.

So let's see what the CLM results are for this. I believe this
does remove the security vulnerability.

Issue-ID: POLICY-1127
Change-Id: Ib659eddc2808131c8dded3782fcf1d382060fc29
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Fixes for oparent upgrade

Clean up mariadb so that it is consistent, allow jetty to be
defined by oparent, and fixes to fully support google guava
upgrade.

Issue-ID: POLICY-1126
Change-Id: Ic0c2fba5f6f5f1a41e53770b95861606a5b4bbe6
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Fixing issue with validation errors"

Merge "Remove override of parent aaf version"

Merge "Fix Policy GUI Template Issue"

Fixing issue with validation errors

I found this change internally that was committed to fix an issue with validating required
fields when creating some Config MS policies.  This should fix the CSIT tests
and the pre-loaded policies for SDNC Naming policies

Change-Id: I04406b8efbef173a6ab967a32cfbd86e17b7c59e
Issue-ID: POLICY-983
Signed-off-by: Michael Mokry <mm117s@att.com>

Fix Policy GUI Template Issue

Fix the a bug in this html file by adding msPolicyloader

Issue-ID: POLICY-1118
Change-Id: I7a45b232ed0680f1c8233b8ed71fdde685fbb9c9
Signed-off-by: Wang, Frank(gw1218) <gw1218@att.com>

Remove override of parent aaf version

policy/parent now declares the aaf dependency so consistency
is maintained in all the repositories. Removing the version
declaration in order to pull in the declared version.

The constructor changes to add a namespace. A null value is
still possible to be used. If we are given a namespace then we
will have to add that in. Preferrably via a properties file.

I also noticed there is a hard-coded URL in here in the
setupAAF() method on line 212. That should be moved to a property
file.

Issue-ID: POLICY-1119
Change-Id: I97dcd7d669d3d05db7c6597be6f3038183fa4e17
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Added validation on matching required fields"

Merge "AAF integration in Policy SDK"

Added validation on matching required fields

Import policy should report to end users if there are missing required
matching fields in policy template.

Issue-ID: POLICY-1117
Change-Id: Iac4485259b73784e641f42a22d5a12c524f4b8e2
Signed-off-by: Wang,Frank(gw1218) <gw1218@att.com>
Signed-off-by: Wang, Frank(gw1218) <gw1218@att.com>

AAF integration in Policy SDK

Upgraded SDK version.
Added SQL script with encrypt key.
Added a function to save role on reading from session in
PolicyController.java
Addressed Check Style warnings in PolicyController.java

Issue-ID: POLICY-1088
Change-Id: I446b6c1a0db2a989ad3f4e982e5cdba845f81045
Signed-off-by: rb7147 <rb7147@att.com>

Upgrade elasticsearch v6.3.0

Resolves a security issue around information disclosure. The
recommendation is to upgrade to this version for the fix.

Issue-ID: POLICY-1063
Change-Id: Idebb19d2c530b716a25aa91832c053e6b2d0ffce
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Upgrade commons-compress v1.18"

Fixed incorrect model name

Changed the model name of the script to be the correct SDNC-GenerateName
version CSIT to fix the failed CSIT test cases

Change-Id: I8974202cfc8cea3a19e102813fbf2a4f449cd11b
Issue-ID: POLICY-983
Signed-off-by: Michael Mokry <mm117s@att.com>

Upgrade commons-compress v1.18

Fixes 2 security issues and brings the common property
value to the root pom.xml for easier management.

Issue-ID: POLICY-1063
Change-Id: I56e3318bdfae2d205c61a5ad7af70a79bd70dada
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Upgrade springframework 4.3.18.RELEASE

To resolve security issue for 4.3.15.RELEASE and also
have a single place where the dependency is defined for
the sub-modules.

NOTE: 4.2.0.RELEASE is left alone in Policy SDK App
due to Portal SDK dependency.

Issue-ID: POLICY-1063
Change-Id: Ibdc464b16b4771faccb2c8c472efa7fbcbf3352c
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Upgrade dmaap client"

Upgrade dmaap client

Upgrading to dmaap 1.1.8 for Casablanca. Hoping they have
removed some license issues.

Also, policy/parent has the version property definition
now because this dependency is shared across multiple repos.

Issue-ID: POLICY-1116
Change-Id: I6ffe7b7cf4b46b12b3ec8d534a58f0be40bb97b9
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Added SQL to import SDNC naming model

Modified the script to inlcude an insert statement to add the
SDNCGenerate Naming model to the microservicemodels table during
installation

This will need to be merged before I add the CSIT test to Integration

Change-Id: I1620ce3f1097d45111bddb08c12b5b8c12bb1b2e
Issue-ID: POLICY-983
Signed-off-by: Michael Mokry <mm117s@att.com>

Modified Min Max Guard xacml template

Changed the condition to remove not from permit and have only one not
for deny

Change-Id: Ie4563aba000fa5366095fe23d23c37019b5224d4
Issue-ID: POLICY-1038
Signed-off-by: Michael Mokry <mm117s@att.com>

Merge "New min/max Guard Policy"

New min/max Guard Policy

Made changes to support Guard policy create/update and other policy manager
functions including deletePolicy and pushPolicy.

Includes changes for new Guard policy support using GUI and API (Rest and Java Client)

Made changes to address Jim's comments

Made changes to address Liam's comments

Change-Id: I133fe1fd9287ea77ea41a2788de90c7642c36b6a
Issue-ID: POLICY-1038
Signed-off-by: Mike Mokry <mm117s@att.com>

Change websockets to managed version

Change-Id: Iff8f9053640752f86e6f5aa94c1f16f6d3e01d6a
Issue-ID: POLICY-884
Signed-off-by: liamfallon <liam.fallon@ericsson.com>

Resolved XACML Platform Sonar issues

Issue-ID: POLICY-902
Change-Id: If22627dfe7df993b5c725ddf3a66c10ebbaa73ca
Signed-off-by: rb7147 <rb7147@att.com>

Merge "XACML Platform Enhancements"

Merge "TOSCA model enhancements"

XACML Platform Enhancements

To Support Raw Policy Creation from GUI and API.
GetDecision on combining algorithim with Policy Set.

Issue-ID: POLICY-902
Change-Id: Ie3189f8ded2e03366bc7d65d15b95b88c89b0acd
Signed-off-by: rb7147 <rb7147@att.com>

TOSCA model enhancements

Add those features below to existing TOSCA Policy model:
1. Set default value to HTML element
2. Set description of a property as a tooltip
3. Support Boolean type
4. User defined object can be re-used by different properties in the same yaml file

Issue-ID: POLICY-900
Change-Id: I337107828ce8130b9a68a2ba000ee719bbcb923e
Signed-off-by: Wang,Frank(gw1218) <gw1218@att.com>

Upgrade to policy/parent

Change from oparent to policy/parent which inherit from
the latest oparent. The latest oparent has dependencyManagement
recommendations from the ONAP Security sub committee that
the policy team is adhering as part of the effort to consolidate
security work across projects.

Change-Id: I1a43fc5a04dbc8c4cb2b7bedbca67ad3e9dd120b
Issue-ID: INT-619
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>

Merge "Enhancement on Dashboard"

Modified Blacklist documentation

Modified Blacklist documentation
-------------------------------------------------------------

Issue-ID: POLICY-947

Change-Id: Ie31d45c302c51abc09705df8e714fb42d9ced1a6
Signed-off-by: Saryu Shah <ss3917@att.com>

Enhancement on Dashboard

Enhancement on dashboard by including more info related to CRUD actions
on policies and also providing an advanced search feature.

Issue-ID: POLICY-1042
Change-Id: I987e0ca13824413f90299a366fb4464fca7de06a
Signed-off-by: Wang,Frank(gw1218) <gw1218@att.com>

policy/engine changes to support PE https

Enabled https connector in server.xml and modified the docker-install.sh
to fix the default variable name for policy truststore

Change-Id: I6db6a6d2353130c91c1f344ea7927d187150adca
Issue-ID: POLICY-781
Signed-off-by: Michael Mokry <mm117s@att.com>

Merge "Use builder for std pap policy"

Merge "Updated guard blacklist documentation"

Use builder for std pap policy

Modify constructor in stdpappolicy to use builder
this will fix the sonar issue related with more than
7 parameters method parameters

Issue-ID: POLICY-1016
Change-Id: Id736a66e5ea48f0f233cffb640cdce8aaa8f2c0d
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>

Updated guard blacklist documentation

Updated guard blacklist documentation
-------------------------------------------------------------

Change-Id: Ib99fd83119994a59e9708e3585d0eafdababc8d5
Issue-ID: POLICY-947
Signed-off-by: Saryu Shah <ss3917@att.com>

https certs with aaf+pe containers compatibility

Change-Id: I21ed7a0fea5ea7d62857a077fa2568da4af99d26
Issue-ID: POLICY-1057
Signed-off-by: Jorge Hernandez <jh1730@att.com>

Merge "Junit Coverage for Import Black List Entries"

Junit Coverage for Import Black List Entries

Issue-ID: POLICY-901
Change-Id: I3776d3c4ed01bb27ed930c074c370361e33b100a
Signed-off-by: rb7147 <rb7147@att.com>

Merge "Resolved the Sonar issues"

Resolved the Sonar issues

Issue-ID: POLICY-901
Change-Id: I30e1d075dd8eb4739bcb5aa1f2503a016900ff75
Signed-off-by: rb7147 <rb7147@att.com>

Optimize PAP policy constructor with builder

Fix sonar issue about constructor using more than 7 params.
Use builder parameters object for setting the attributes of
std pap policy.Recommit1

Issue-ID: POLICY-1016
Change-Id: Id9631a42813960baa78df4ab0c0f6706244c3180
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>

Merge "Modify std pap policy to use builder in constr"

Merge "Fix comparison issues in onap pap rest"

Modify std pap policy to use builder in constr

Fix sonar issue about constructor using more than 7 params.
Use builder parameters object for setting the attributes of
std pap policy.Rework2

Issue-ID: POLICY-1016
Change-Id: If26f7daa8d5b5d387238aad74daeddd49a8b95bb
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>

Fix comparison issues in onap pap rest

Fix issues about incorrect comparison in pap rest modules
indicated by analysis tools.Rework to fix review
comments1

Issue-ID: POLICY-1016
Change-Id: I7a0ee45a721307fc3ba89ab65e807613a3df2542
Signed-off-by: krisjinka <kris.jinka@samsung.com>

Add PAP Policy parameter builder xacml policy

Fix Sonar flagged major issue about 14 params instead of 9
in the constructor of std pap policy class. Commit5 for
modify constrctor.fix test

Issue-ID: POLICY-1016
Change-Id: I79c8482b7fc4e203cb18402ba2b5a68a120c57b1
Signed-off-by: krisjinka <kris.jinka@samsung.com>

Fix sonar issue with constrctr in xacml comp

Fix sonar issue of Method has 9 parameters, which is
greater than 7 authorized in StdPDPPolicy. Make the
actual sonar fix

Issue-ID: POLICY-1016
Change-Id: Ibc79ddce81a6209ec190c7f393557e9c569df453
Signed-off-by: krisjinka <kris.jinka@samsung.com>

Resolve sonar issue in Policy parameters

Fix sonar issue of Method has 9 parameters, which is
greater than 7 authorized in PolicyParameters class
and related test class.Rework2

Issue-ID: POLICY-1016
Change-Id: I3096566d958e093439bcc95d92f1fe92e0ac57be
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>

Modify Policy manager servlet for sonar issues

Fix basic sonar issues in policy manager servlet
major and minor issues reported by sonar
Basic refactoring

Issue-ID: POLICY-1016
Change-Id: If3c19943c04d9af43807090b6340547d69ca6a19
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>

Merge "Modify ONAP PAP REST classes basic checkstyle"

Merge "Fix major sonar issues in SDK APP admin mod"

Modify ONAP PAP REST classes basic checkstyle

Modify tabs to space for checkstyle issue fix in ONAP PAP REST
module. It helps future reviews and refactoring. Remaining classes

Issue-ID: POLICY-1014
Change-Id: I677eadac9e8364b388be922024528c27fd1e2545
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>