nifi-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From alopresto <...@git.apache.org>
Subject [GitHub] nifi pull request #2983: NIFI-5566 Improve HashContent processor and standar...
Date Tue, 04 Sep 2018 01:40:37 GMT
GitHub user alopresto opened a pull request:

    https://github.com/apache/nifi/pull/2983

    NIFI-5566 Improve HashContent processor and standardize HashAttribute processor

    ### This PR should not be merged before [PR 2980](https://github.com/apache/nifi/pull/2980)
    
    This PR introduces two new processors `CryptographicHashContent` and `CryptographicHashAttribute`
(renamed from `CalculateAttributeHash` in PR 2980), deprecates `HashContent`, and adds documentation
to `HashAttribute` indicating the new processor performs the expected behavior. 
    
    Thank you for submitting a contribution to Apache NiFi.
    
    In order to streamline the review of the contribution we ask you
    to ensure the following steps have been taken:
    
    ### For all changes:
    - [x] Is there a JIRA ticket associated with this PR? Is it referenced 
         in the commit message?
    
    - [x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying
to resolve? Pay particular attention to the hyphen "-" character.
    
    - [x] Has your PR been rebased against the latest commit within the target branch (typically
master)?
    
    - [ ] Is your initial contribution a single, squashed commit?
    
    ### For code changes:
    - [x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check
clean install at the root nifi folder?
    - [x] Have you written or updated unit tests to verify your changes?
    - [ ] If adding new dependencies to the code, are these dependencies licensed in a way
that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?

    - [ ] If applicable, have you updated the LICENSE file, including the main LICENSE file
under nifi-assembly?
    - [ ] If applicable, have you updated the NOTICE file, including the main NOTICE file
found under nifi-assembly?
    - [ ] If adding new Properties, have you added .displayName in addition to .name (programmatic
access) for each of the new properties?
    
    ### For documentation related changes:
    - [x] Have you ensured that format looks appropriate for the output in which it is rendered?
    
    ### Note:
    Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/alopresto/nifi NIFI-5566

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/nifi/pull/2983.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2983
    
----
commit e2cf56994edd3c72baa346c46c4fab2c03a654fa
Author: Otto Fowler <ottobackwards@...>
Date:   2018-06-18T15:00:38Z

    NIFI-5147 Add CalculateAttributeHash processor

commit 81d7e794e11855b0d44521c55baaae0771c9de44
Author: Otto Fowler <ottobackwards@...>
Date:   2018-06-26T18:51:06Z

    add warning and recommendation link

commit 808b9b22e4137ce26e7361015a4953dacc0296a1
Author: Otto Fowler <ottobackwards@...>
Date:   2018-07-01T16:18:08Z

    per review
    
    - added properties to control behavior when attributes that are configured are partially
or completely missing
    - set charset with a property
    - added tests

commit a05de1c62710acf7f91855d5492f01e2ed46e155
Author: Otto Fowler <ottobackwards@...>
Date:   2018-07-01T16:25:24Z

    fix assert parameter order

commit 66577ddb57ae4fff198608d29ed4490b1543e424
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T02:26:53Z

    NIFI-5147 Added HashAlgorithm enum for CalculateHashAlgorithm and HashContent.
    Added unit tests.

commit 8a594e5b1ec9f189a14d933e7e586e9bc1e04432
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T03:07:18Z

    NIFI-5147 [WIP] Used HashAlgorithm enum in CalculateHashAttribute.
    Cleaned up typos and descriptions.
    Added unit test demonstrating missing Blake2 algorithm.

commit 39e9cd2325f5f2312daa19d1ccb66664128de384
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T03:18:44Z

    NIFI-5147 [WIP] Added logic for Blake2 algorithms (needs refactoring).

commit 68db917487c009eb9a3b75ed50d174f809c51e3a
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T03:31:27Z

    NIFI-5147 Blake2b limited to 160, 256, 384, and 512 in Bouncy Castle implementation.
    Updated unit test.

commit 5165a41bdb90e32e0e81ae402cbff8a61669dd8b
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T03:34:28Z

    NIFI-5147 Finished implementing Blake2 logic.
    Unit test for all default test vectors passes.

commit 6f91e6f9ec3cbbceac5609ca9d495ec259dbc517
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T03:44:10Z

    NIFI-5147 Added unit test demonstrating empty values fail.

commit afd6a0cbf4d1b09f8109150051bd28a90d4fda6a
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T03:45:08Z

    NIFI-5147 Implemented logic for empty input values.

commit 9a47353adc439ca6c6ed4976dfb8327450400f74
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T04:17:54Z

    NIFI-5147 Implemented HashService.
    Added unit tests.

commit 82a213904bcf81845b21ef35a41849b7564d2475
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T04:31:46Z

    NIFI-5147 Implemented unit tests for test vectors, convenience methods, character encoding,
and defaults.

commit b0f8f23d7b4e79d7aa9c635f7f5e6e61f6dade6a
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T04:33:01Z

    NIFI-5147 Refactored CalculateAttributeHash to use HashService.
    All unit tests pass.

commit e0847fb6e09d494536f7fef83fe75656a977144e
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T05:01:14Z

    NIFI-5147 Added #clearProperties() to TestRunner, StandardProcessorTestRunner, and MockProcessContext.

commit 040f14ebac33501e5f4b630dca9f007d93adcfef
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T05:46:59Z

    NIFI-5147 Added HashAttribute#fromName() method to be more generous on name matching.
    Added unit test.

commit 73e7142d4d1693c9aba7eed59469c6c495a9538f
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T06:19:21Z

    NIFI-5147 Handled edge cases.
    Added unit tests.

commit 1d45d68952b41224b194733fa47ea75bf417340b
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T06:19:51Z

    NIFI-5147 Removed old unit tests.

commit f0281aa94fa41b419cafe1e94fe60a1ba71f2696
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T06:59:02Z

    NIFI-5147 Fixed checkstyle issues.

commit 4dd248f579a4eaea9f45e9c7fb7ba15cd0113a01
Author: Andy LoPresto <alopresto@...>
Date:   2018-08-31T07:16:16Z

    NIFI-5147 Added full algorithm description to AllowableValue.

commit 36acaaffeb359d07d8ae234248a4755d982620e4
Author: Andy LoPresto <alopresto@...>
Date:   2018-09-01T02:28:49Z

    NIFI-5147 Added buildHashAlgorithmAllowableValues() and hashValueStreaming() methods to
HashService.
    Added unit tests.

commit 8d882c9713ddcb24f89aee8f9f8b78412f6d3b9b
Author: Andy LoPresto <alopresto@...>
Date:   2018-09-01T03:30:05Z

    NIFI-5566 Moved HashContent to HashContentLegacy.
    Moved TestHashContent to TestHashContentLegacy.
    Implemented "modern" HashContent.
    Added unit tests as HashContentTest.
    Updated processor manifest.
    Updated Javadoc.

commit b4471c7a5ed658f9e542e62747c656f8cdec9e65
Author: Andy LoPresto <alopresto@...>
Date:   2018-09-01T04:12:48Z

    NIFI-5147 Moved HashAttribute to HashAttributeLegacy.
    Moved TestHashAttribute to TestHashAttributeLegacy.
    Moved CalculateAttributeHash to HashAttribute.
    Moved CalculateAttributeHashTest to HashAttributeTest.
    Updated processor manifest.
    Updated Javadoc.

commit 9d243e9db6e11e3507b50fc241c6883c4f18bc54
Author: Andy LoPresto <alopresto@...>
Date:   2018-09-04T01:34:30Z

    NIFI-5566 Renamed new components to CryptographicHashAttribute and CryptographicHashContent.
    Added documentation about deprecated/renamed components.
    Fixed unit test resource path.

----


---

Mime
View raw message