Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] Encrypt/Decrypt template credentials #216

Merged
merged 1 commit into from
Nov 29, 2023

Commits on Nov 29, 2023

  1. Encrypt/Decrypt template credentials (#197)

    * added RegisterRemoteModelStep and tests
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Adding RegisterLocalModelStep, fixing tests, adding input/ouput definitions to workflow step json
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Fixing javadoc warnings, fixing log message
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Addressing PR comments,making description field optional for RegisterRemoteModelStep and RegisterLocalModelStep
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * moving modelConfig builder before adding allConfig
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * initial implementation
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Fixing create workflow transport action
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Removing duplicate register_remote_model validator
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Adding bouncy castle dependency to resolve encryption issue
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Fixing CreateWorkflowTransportActionTests
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Adding initial unit tests for encryptor utils
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Implemented encryption/decryption for workflow node user inputs with credential
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Addressing PR comments
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Suppressing unchecked warning, making credential strings constants
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Removing setMasterKey from initializeMasterKey method
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Adding final template encryption decryption test
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Addressing PR comments, changing master key index name to config, fixes error messages as well. Added create time field to config index, ensured that updates are also encrypted
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Added TODO
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * changing getMasterKeyIndexMapping method name
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    * Removing unnecessary aws sdk dependency
    
    Signed-off-by: Joshua Palis <[email protected]>
    
    ---------
    
    Signed-off-by: Joshua Palis <[email protected]>
    (cherry picked from commit 772fbbd)
    Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
    github-actions[bot] committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    e7fd1be View commit details
    Browse the repository at this point in the history