Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

[#IOPID-127] Use Crypto utils from fn-commons #35

Merged
merged 1 commit into from
Apr 11, 2023

Conversation

BurnedMarshal
Copy link
Contributor

List of Changes

Upgrade io-functions-commons to v28.1.0
Remove duplicated code
Rename toHash function with sha256 from fn-commons

Motivation and Context

In order to reduce code duplication we have migrated some crypto utils functions in io-functions-commons. Now this function use shared functions intead embedded ones.

How Has This Been Tested?

unit test

Screenshots (if appropriate):

Types of changes

  • Chore (nothing changes by a user perspective)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@BurnedMarshal BurnedMarshal requested a review from a team as a code owner April 11, 2023 11:23
@github-actions
Copy link

Jira Pull Request Link

This Pull Request refers to the following Jira issue IOPID-127

Copy link
Contributor

@gquadrati gquadrati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BurnedMarshal BurnedMarshal merged commit f2a1602 into master Apr 11, 2023
@BurnedMarshal BurnedMarshal deleted the IOPID-127-use-fn-commons-crypto branch April 11, 2023 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants