Fix javadoc formatting for SHA256Calculator

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

Adds documentation to IdmLightConfig

This is a formatting change to add documentation to IdmLightConfig.
This change extracts constants from magic strings.

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

Formatting applied to aaa-authz bundles

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to features"

Merge "Formatting applied to aaa's distribution-karaf bundle"

Merge "Formatting applied to aaa-shiro-act bundle"

Merge "Formatting applied to aaa-shiro bundle"

Formatting applied to features

Removed any tabs and innapropriate whitespace from feature
poms and features.xml files.

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

Formatting applied to aaa's distribution-karaf bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Formatting applied to aaa-shiro-act bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn-sts bundle"

Merge "Formatting applied to aaa-authn-sssd bundle"

Formatting applied to aaa-shiro bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn-store bundle"

Formatting applied to aaa-authn-sssd bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn-mdsal bundles"

Formatting applied to aaa-authn-sts bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn-federation bundle"

Formatting applied to aaa-authn-store bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn-keystone bundle"

Formatting applied to aaa-authn-federation bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Formatting applied to aaa-authn-mdsal bundles

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn-idpmapping"

Formatting applied to aaa-authn-keystone bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn-basic"

Formatting applied to aaa-authn-idpmapping

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Merge "Formatting applied to aaa-authn bundle"

Formatting applied to aaa-authn-basic

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Formatting applied to aaa-authn bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, reorganizing imports, and converting
all line endings to Unix-style.

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

Formatting applied to aaa-authn-api bundle

Fixes flagrant checkstyle violations.  This is done using
eclipse formatting, adding Logger where printStackTrace()
existed, and reorganizing any imports that were in the
wrong order.

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

Merge "Bug 4804 IDMLight REST endpoints do not redact the salt user field"

Formatting applied to aaa-h2-store package

Fixes checkstyle violations.  Deprecates and logger wrapper methods,
in favor of directly invoking LOG methods.

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

Bug 4804 IDMLight REST endpoints do not redact the salt user field

Enforces redacted fields salt fields for IDMLight rest endpoints, since
exposing such information is a plausible security concern.   Abstracts several
helper methods to avoid code duplication.  Cleans up error reporting so error
messages are consistent among operations on the /auth/v1/users REST
endpoint.  Reformats several areas of code that do not align to AAA
checkstyle.  Adds much needed documentation on how to utilize the REST
endpoint.

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

Merge "Formatting applied to aaa-idmlight tests"

Formatting applied to aaa-idmlight tests

Fixes formatting in test scripts.  Fixes PasswordHashTest
to throw Exception in setup instead of try/catch so the
test actually fails.

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

Formatting applied to AAAIDMLightModule

Fixed checkstyle errors in AAAIDMLightModule.

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

Merge "Formatting applied to StoreBuilder"

Formatting applied to StoreBuilder

This change tackles some vital formatting issues with StoreBuilder.  The
following is done to the IdmLightApplication class to promote clarity and conformance
to project style:

1) Add some detailed information to the class comment concerning the default installed
accounts.

2) Add TODO concerning what to do if "sdn" is deleted.  Implies that "sdn" could
be made a mandatory domain.

3) Format in eclipse by clicking "format".

4) Fix any errors reported by "mvn checkstyle:checkstyle".

No actual code or functionality changes were made.  This is strictly a
formatting related patch.

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

Merge "Formatting applied to IdmLightApplication"

Merge "Bug 4809 Deprecate the versionhandler for AAA"

Formatting applied to IdmLightApplication

This change tackles some vital formatting issues with IdmLightApplication.  The
following is done to the IdmLightApplication class to promote clarity and conformance
to project style:

1) Move the class documentation comment to the right location.  Add some
detailed information to the class comment concerning different handlers.

2) Add CONSTANT documentation.

3) Format in eclipse by clicking "format".

4) Fix any errors reported by "mvn checkstyle:checkstyle".

5) Add a TODO to address the mismatch in MAX_FIELD_LENGTH between this
class and the Store classes, which assume 128 not 256.

No actual code or functionality changes were made.  This is strictly a
formatting related patch.

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

Merge "Formatting applied to IdmLightProxy"

Formatting applied to IdmLightProxy

This change tackles some vital formatting issues with IdmLightProxy.  The
following is done to the IdmLightProxy class to promote clarity and conformance
to project style:

1) Add instance member documentation.

2) Add static initialization documentation

3) Add info logging to the "clearClaimCache()" method.

4) Format in eclipse by clicking "format".

5) Fix any errors reported by "mvn checkstyle:checkstyle".

6) Eliminate the "debug" logging method in favor of direct invocation.

No actual code or functionality changes were made.  This is strictly a
formatting related patch.

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

Bug 4809 Deprecate the versionhandler for AAA

AAA currently has a rest endpoint to display version information.  This REST
endpoint is:
1) out of date
2) largely unused to our knowledge
3) should be deprecated

This patch does the following:
1) Formats by clicking "format" in eclipse.
2) Fix any checkstyle violations.
3) Puts the class javadoc comment in the correct place
4) Adds an @Deprecated tag so the interface can be safely removed in Boron

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

Formatting applied to RoleHandler

This change tackles some vital formatting issues with RoleHandler.  The
following is done to the RoleHandler class to promote clarity and conformance
to project style:

1) Move the class documentation comment to the right location.  Add some
detailed information to the class comment concerning how to use the REST
endpoints for RoleHandler.

2) Add function documentation comments explaining their responsibilities.

3) Format in eclipse by clicking "format".

4) Fix any errors reported by "mvn checkstyle:checkstyle".

No actual code or functionality changes were made.  This is strictly a
formatting related patch.

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

Formatting applied to DomainHandler

This change tackles some vital formatting issues with DomainHandler.  The
following is done to the DomainHandler class to promote clarity and conformance
to project style:

1) Move the class documentation comment to the right location.  Add some
detailed information to the class comment concerning how to use the REST
endpoints for DomainHandler.

2) Add function documentation comments explaining their responsibilities.

3) Format in eclipse by clicking "format".

4) Fix any errors reported by "mvn checkstyle:checkstyle".

No actual code or functionality changes were made.  This is strictly a
formatting related patch.

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

Merge "Bug 4766 AAA does not provide AuthN only LDAP capability"

Merge "Pull in dependencies from odlparent"

Bug 4783 SQLException masked my IDMStoreException, making debugging impossible

Adds logging to help actually debug StoreException(s), which are usually
caused by SQLException(s).  Propagates the entire throwable to IDMStoreException
instead of just the message, which is more useful for debugging.

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

Bug 4768 AAA STS ServiceWireTask prone to IllegalStateException: BundleContext is no longer valid

ServiceWireTask is deprecated since it is prone to stale BundleContext.  The
functionality is replaced with an OSGi ServiceTracker and
ServiceTrackerCustomizer pair, which handles AAA activation and service loading
asynchronously.  This patch was inspired by this great work:
https://git.opendaylight.org/gerrit/#/c/31070/

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

Bug 4773 Package uses conflict between authn and restconf

Adds necessary annotation libraries from jsr305.

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

Bug 4766 AAA does not provide AuthN only LDAP capability

Provides a wrapper to JndiLdapRealm (Shiro) for AuthN only LDAP integration.
From a security perspective, certain operators may not want to expose LDAP
grouping information to the controller.

Also, this change adds some more necessary documentation to shiro.ini
concerning configuration of Shiro-based AAA.  The documentation provides
examples of how to set up the different LDAP implementations.

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

Pull in dependencies from odlparent

Drop Jersey dependencies from dependency management (this needs
jersey-test-framework-grizzly2 to be added to odlparent).

Switch from javax.servlet:servlet-api:2.5 to
javax.servlet.javax.servlet-api:3.0.1 (which is also OSGi compliant).

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

Bug 4749 odl-aaa-shiro feature doesn't import some runtime dependencies

Imports the correct runtime dependencies so that ODL's Shiro wrapper
and Filtering works.  Also updates to use ${project.version} in the
appropriate places.

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

Modify AAAIDMLightModule to use a ServiceTracker

This replaces the thread that was spawned to busy wait for the IdmStore
service.

Change-Id: Ib90a92716725a7e9c94945f067f5029d0f06526d
Signed-off-by: Tom Pantelis <tpanteli@brocade.com>

Fix Bug 4723 StoreBuilder init times out

Removes StoreBuilder initialization call from IdmLightApplication constructor
to avoid blocking the main OSGi thread.  A separate thread called
StoreServiceLocator, which is spawned during IDMLightModule creation,
is used to track status of IIDMStore, the "pluggable" backing data store for
AAA identity management.  When the store is resolved, the thread invokes
StoreBuilder initialization code to handle initial creation of identity data.

Change-Id: I7cb91f2ed71712932e33188e3b4f6d2b9b0f1aa3
Signed-off-by: Sharon Aicler <saichler@cisco.com>
Signed-off-by: Ryan Goulding <ryandgoulding@gmail.com>

Revert Bug 4723 AAA StoreBuilder init times out too early

This reverts commit 0f25e8c273c6297e4e88f8e2f335e08d0194c029.

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

Merge "Bug 4741 AAA ServiceWireTask causes long running singleFeatureTest"

Bug 4741 AAA ServiceWireTask causes long running singleFeatureTest

Sets StoreServiceLocator thread to be a daemon thread.

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

Bug 4741 AAA ServiceWireTask causes long running singleFeatureTest

Makes ServiceWireTask thread daemon so it won't block.

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

Bug 4732 Convert IDM Light to use config subsystem

Change-Id: I3800b15c4371c5acb7c9ae9b2af0c85412224d72
Signed-off-by: Sharon Aicler <saichler@cisco.com>

change log message from 30s to 10m

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

Merge "Bug 4723 AAA StoreBuilder init times out too early"

Bug 4723 AAA StoreBuilder init times out too early

AAA StoreBuilder will timeout if the backing data store isn't
loaded in 30s.  This is too short for some features.

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

Merge "Add Deprecated Tag to TokenAuthFilter"

Add Deprecated Tag to TokenAuthFilter

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

Wiring Service for IDM Light

Change-Id: I644e16f137862a20febdbec3b69912fb23e8c9f4
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Inherit version from parent

Since the bundle is using the same version as the parent version we can
inherit it and clean up the pom a little. Some aaa artifacts are already
using this method so this patch just makes the remaining ones update for
consistency.

Change-Id: I7af809d24475f096fbcedacbed1f239ff76fecec
Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>

Add the new config artifact to feature dependencies.

AAA distribution job (and several downstream merge jobs)
are failing because they are unable to locate
org.opendaylight.aaa:aaa-h2-store:xml:config:0.3.0-SNAPSHOT
This is an attempt to fix that.

Also, the new config artifact is added to artifacts.

Change-Id: Id37064d6eb3eaf87778c0f7f3cec7cd9dcb463a3
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

Merge "Revert "Revert "BUG 4624: Use default domain when credentials dont provide one"""

Revert "Revert "BUG 4624: Use default domain when credentials dont provide one""

This reverts commit 7bc9064d9fca906e397afadef2ab86838402ceb0.

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

H2 Store converted to Config Subsystem

Change-Id: I6400587b920f8558512b942889216f3c72e42a49
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Hardwire Services in sts ServiceLocator

Change-Id: Ia408c78484ee0183626437318c147d60979d204f
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Apply comments for MDSAL Store

This reverts commit 7050d9954aed918d6b6bca4ccaec803a33f83a1d.

Change-Id: I450370f72888a54790d2c3e1077adab96a22bc40
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Re-apply MD-SAL Data Store For AAA

This reverts commit 56837b89838429e326b8f18312f66331fe85eb44.

Change-Id: Ic8ec33213fbd8a632ab4b37a25fb1266f2f35d4f
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Merge "Add OAuth2 Token to Shiro to maintain cross-version compatibility"

Add OAuth2 Token to Shiro to maintain cross-version compatibility

Adds a mechanism to authenticate OAuth2 tokens, similar to
TokenAuthFilter's implementation.

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

Revert "MD-SAL Store for IDM"

This reverts commit 4a4dcdb561819b4016b918516d3dddf116016d0b.

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

Merge "Revert "BUG 4624: Use default domain when credentials dont provide one""

Merge "Revert "MD-SAL Store applied Comments""

Revert "BUG 4624: Use default domain when credentials dont provide one"

This reverts commit 4e67c246552d8bb40d5090bc0f572df91cce84ac.

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

Revert "MD-SAL Store applied Comments"

This reverts commit f429ddf8d4db71c40ca9957c52e4052b95cb10de.

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

BUG 4624: Use default domain when credentials dont provide one

Change-Id: I28a76301b2a588dd52d686d76f7abcace036ee88
Signed-off-by: Tomas Cere <tcere@cisco.com>

Merge "Enables TokenAuthRealm adapter to existing data stores"

Merge "MD-SAL Store applied Comments"

MD-SAL Store applied Comments

Change-Id: Idc25ccc288767ee7d97681734a5a16b78689729f
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Enables TokenAuthRealm adapter to existing data stores

Enables a new realm to check authentication against existing data
stores (IdM & MD-SAL).

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

Switch DefaultRBACRules to utilize lazy initialization

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

Bug 4588 aaa-authn-api causing stack overflow for sfc tests

Swaps out the logging library in favor of the l2switch library, so
that the classpath is not polluted with APIs that do not behave
well together.

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

Merge "MD-SAL Store for IDM"

MD-SAL Store for IDM

Change-Id: Ib44bf394616b5a471025a1d5b98ea3574f2e94f3
Signed-off-by: Sharon Aicler <saichler@cisco.com>

Fix aaa-shiro JDK8 compatibility

Fixes how external links are documented.

Change-Id: I493476675aa7b4f36ca82d33c0469da5b1605fc7
Signed-off-by: Robert Varga <rovarga@cisco.com>

Remove duplicate dependency

Maven warns about multiple entries in dependecies -- remove them to fix
the warning.

Change-Id: Ic1cfa942dd44fee9a17cf09020a5cd98b0f0547f
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Fix JDK8 compatibility"

Fix JDK8 compatibility

I355acae1ac6652eb15cbd83ce9b0fcdcb9cc8afc caused compilation regression
in CredentialAuth. Fix that.

Change-Id: Ic523cf60dbafc681962d19c7cdb67b17cef69f89
Signed-off-by: Robert Varga <rovarga@cisco.com>

Merge "Revert "Bug 2515 - Jersey to support JAX-RS 2.0""

Revert "Bug 2515 - Jersey to support JAX-RS 2.0"

This reverts commit b1c7e0948abe0d62f130667ee98fd89b148a1009.

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

Merge "Bug 2515 - Jersey to support JAX-RS 2.0"

Remove unnecessary synchronization locks in service locator

ServiceLocator instance variables are already volatile to ensure
consistency across threads.  Thus, synchronization locks aren't
needed, since reads and writes are already atomic.

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

Merge "DataEncrypter Unit Test and remove previous changes"