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>

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>

Fix major sonar issues in SDK APP admin mod

Fix Sonar reported major issues regarding extracting constants,
method complexity, duplicated code blocks, more than 3 nested
control blocks. Rework3 after self review.

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

Convert tabs to space in ONAP PAP REST1

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

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

Merge "Copy policy-endpoints from drools-pdp to common"

Merge "Convert tabs to spaces basic refactoring"

Merge "Fix major sonar issues in SDK APP"

Convert tabs to spaces basic refactoring

Change tabs into spaces for checkstyle fixes in ONAP REST module of
policy engine. These changes come in way for review. Hence doing them
before refactoring is better.

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

Fix major sonar issues in SDK APP

Fix major issues reported by sonar in POLICY SDK APP files such as
CreateBRMSParamController, PolicyManagerServlet. Also modify tabs to
space. Rework2 based on review comments

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

Copy policy-endpoints from drools-pdp to common

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

Fix Sonar issues in utils PDP ParseLog rework

Fix Major Sonar issues in ParseLog BackupMonitor and ONAP-PDP component.
Issues related to nested control statements. Rework2 based on review comments

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

Merge "Copy policy-endpoints from drools-pdp to common"

Merge "Fix nested stmts issue of Sonar"

Merge "Refactor code for nested stmts in policy std"

Copy policy-endpoints from drools-pdp to common

Replaced references to classes deleted from drools-pdp with references to
the corresponding classes in polcy-common

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

Refactor code for nested stmts in policy std

Refactor code for nested control statements in PolicyEngineAPI policy.std
package. These issues are flagged by Sonar as Major issues. Rework based on
review comments

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

Fix nested stmts issue of Sonar

Policy rest util MSModeUtils have Sonar flagged issues about nesting
more than 3 control stmts. Fix these. Also a minor change from prev
commit about BrmsPush.java. Rework based on review

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

Merge "Fix issues reported by sonar"

Fix issues reported by sonar

Fix issues related to unused method variables, extract constants,
not nest more than 3 control stmts, use try with res in policy
engine project. Rework for comments

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

Merge "Decision BlackList Guard Enhancements"

Merge "Fix sonar issue about nested stmts"

Fix sonar issue about nested stmts

Refactor code to not nest more than 3 condition or loop stmts
in policy PolicyEngineUtils. Rework based on review comments

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

Decision BlackList Guard Enhancements

While creating a decision Bl Guard Policy we are allowing to add
Blacklist entries through file upload for bulk from GUI.

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

Merge "Refactor PolicyNotificationMail for sonar issues"

Add modified copyright notice

For the files which are modified add
modified copyright notices as per comments from earlier review

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

Refactor PolicyNotificationMail for sonar issues

SONAR flagged issues related with complexity of method being high
and more than 3 nested loops in the PolicyNotificationMail

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

FIX SONAR ISSUE NESTED TRY BLOCKS

refactor the code for nested try blocks and rework
based on comments and fix nested loops complexity

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

Upgrade tomcat to clear security issues

New security issue has appeared, so we will do minor upgrade.

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

USE TRY WITH RESOURCES

Fix sonar issues to convert try blocks to
try with resources plus change tabs 2 space

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

Reverting "Merge "fix try block""

This reverts commit 240bbdbdc76e0e91af9f18164eadb2708d14c6fb, reversing
changes made to 3ee69f1af0acddd1e06060ad88c0029fe7ceeaaf.

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

Merge "fix try block"

fix try block

Issue-ID: POLICY-961

Change-Id: Ia1f388368007a4a82a57520dc6ddd99cc484a393
Signed-off-by: Krishnajinka <kris.jinka@samsung.com>

Upgrade Portal SDK to resolve Security Issues

Upgraded version in pom to 2.3.1
Added SQL scripts from Portal to Upgrade Portal related db tables
Added SQL downgrade script and license header
Sorted SQL commands in upgrade script by function_cd per Jim

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

Revert "Fix policy gui missing schema_info table"

This reverts commit e67481ed648c419a1645d0fc8b98506fe544a454.

Change-Id: I092053ec1815cf7ca598b1913663532673125604
Issue-ID: POLICY-898
Signed-off-by: Joseph Chou <jc2555@att.com>

Merge "Address missing files in junit tests"

Merge "Don't use hard-coded tmp path"

Merge "Fix console warning about xacml.properties"

Address missing files in junit tests

Renamed logback.xml to logback-test.xml.
Added policyLogger.properties to config directories.

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

Remove %h from log file names

Modified code, either removing "%h" or replacing it with a directory
name that is relative to the junit test.

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

Fix console warning about xacml.properties

Configured location of property file.

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

Don't use hard-coded tmp path

Modified code to use java system variable to identify temp
directory.

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

Change PrefixList.class to Zone.class

Changed the class.  Modified the junit test to catch that type
of issue, refactoring it to extract common code.

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

Fix policy gui missing schema_info table

Modify docker-install.sh to include db schema update

Change-Id: I75941522ea932e59cd85d7ff9887bf7d4759ae7c
Issue-ID: POLICY-898
Signed-off-by: Joseph Chou <jc2555@att.com>

Merge "Set default logger to EELF instead of SYSTEMOUT"

Use POLICY_LOGS env variable in logging.properties

Some logger.properties have /var/log/onap hard-coded; changed those
to use $POLICY_LOGS.
Other logger.properties had ${POLICY_LOGS} instead of ${{POLICY_LOGS}};
fixed those, too.

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

Set default logger to EELF instead of SYSTEMOUT

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

Merge "Enforce logback version"

Enforce logback version

Some of the projects are being built with logback version, 1.1.3, which
does not support the SizeAndTimeBasedRollingPolicy appender.  Updated
the offending pom.xml to force it to use logback 1.2.3 instead.

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

Merge "Decryption messages should not be errors"

Merge "Remove "&quot;" from logback xml files"

Remove "&quot;" from logback xml files

Logback complains if %d{} values contain "&quot;", but it works fine
without them, thus they have been removed.
Update licenses.

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

Decryption messages should not be errors

Modified code to generate INFO messages when decryption fails,
instead of ERROR or WARN.  Also modified some of the text in
the afforementioned messages.

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

Fixed Model Content Validation for API

This fixes the validation of the content of the model to include
validating parent nodes that are required based on the yaml. Prior to
this we were only validating required child nodes and in some cases this
was causing failures to create a policy.

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

Doc Updates: Policy OOM; StateMgmt

Updated policy documentation: Policy OOM; StateMgmt
-------------------------------------------------------------

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

Update engine to SNAPSHOT-1.3.0

Snapshot updated for Casablanca.

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

Document Security section of the Release Notes

Change-Id: I2b74eb994ec16f47654e84a43a066c3497caca62
Issue-ID: DOC-271
Signed-off-by: Gildas Lanilis <gildas.lanilis@huawei.com>

Update Release Notes

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

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

Sonar fixes for TOSCA Parser

I fixed the sonar blocker in regards to nullPointerException and few
other majors that I found and were easy to fix.  I ran my local sonar
and the blocker was removed after the changes.

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

Merge "Replace LOGPATH with LOGSUFFIX"

Merge "Fix Issues in TOSCA Parser"

Fix Issues in TOSCA Parser

Fixed a bug and added some new validations in TOSCA Parser

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

Replace LOGPATH with LOGSUFFIX

docker-install.sh does not expand ${{xxx}} values in .conf
files before using them to update target files.  Consequently,
replaced LOGPATH with LOGSUFFIX in .conf files and updated
target files to use ${{POLICY_LOGS}}${{LOGSUFFIX}} instead.
Replace "${POLICY_HOME}/logs" in auxiliary files (e.g., logrotate)
with "${POLICY_LOGS}/policy".
Update license dates.
In .sh files, changed ${{POLICY_LOGS}} to ${POLICY_LOGS}, and
ensured that env.sh is sourced.
Use hard-coded paths in conf files instead of replacing
LOGPATH with LOGSUFFIX.
Update license dates.
Repoint path for IntegrityMonitor.log, as we cannot easily move
since it's in common.
Replace hard-coded log path in docker-install.sh.
Undo hard-coded log paths where possible, and modify scripts to
pass $POLICY_LOGS to JVM.

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

Updated version/date in release notes

Updated version/date in release notes
-------------------------------------------------------------

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

Merge "Update yml to use $POLICY_LOGS"

Merge "Update Policy Engine to 1.2.3-SNAPSHOT"

Update yml to use $POLICY_LOGS

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

Updated release notes

Updated release notes
-------------------------------------------------------------

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

Update Policy Engine to 1.2.3-SNAPSHOT

These reviews must be merged in order because of layering.

This review should be merged FOURTH.

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

Add braces around property name

In pdplp.conf, changed ${POLICY_LOGS} to ${{POLICY_LOGS}}.

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

Merge "Upgrade SNAPSHOT to 1.2.2"

Merge "Added Release Notes - Known Issues"

Added Release Notes - Known Issues

Added Release Notes - Known Issues
-------------------------------------------------------------

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

Upgrade SNAPSHOT to 1.2.2

These reviews must be merged in order because of layering.

This review should be merged SECOND

Note: In order to get past the following layering problems, there
are temporary chages to the following POMs to temporarily use 1.2.1-SNAPSHOT
for those dependencies. They are
ONAP-PAP-REST/pom.xml
 org.onap.policy.drools-applications.controlloop.common.policy-yaml
ONAP-PDP/pom.xml
 org.onap.policy.drools-pdp.policy-endpoints

A separate review will be used to restore these dependencies to 1.2.2-SNAPSHOT
once the drools-pdp and drools-applications reviews are submitted and merged

PLD added version.properties

Issue-ID: POLICY-798
Change-Id: Id7e83f2ba12181c63cc8845b7eeccf2004d2bab9
Signed-off-by: liamfallon <liam.fallon@ericsson.com>
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
Signed-off-by: liamfallon <liam.fallon@ericsson.com>

Merge "policy documentation - release notes"

policy documentation - release notes

policy documentation - release notes
-------------------------------------------------------------

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