Skip to content

feat(aes): 💅 improve encryption/decryption with Strings#14

Merged
felipebonezi merged 1 commit into
mainfrom
feature/refactor-icipher
Oct 21, 2022
Merged

feat(aes): 💅 improve encryption/decryption with Strings#14
felipebonezi merged 1 commit into
mainfrom
feature/refactor-icipher

Conversation

@felipebonezi

@felipebonezi felipebonezi commented Oct 21, 2022

Copy link
Copy Markdown
Owner

Verifique se o PR cumpre os seguintes pontos

  • Testes para as alterações foram adicionados.
  • Documentação foi adicionada ou alterada.

What's Changed

✨ New feature

  • Fixes Problems to encrypt and decrypt when uses encryptToString method #3
    • We were using Strings as primary response from encryptToString but String is not a good container for binary data. Specifically, such a string will almost surely contain unprintable characters, so if you print it or try to pass it via HTML or some other means, you will get some problems (data loss, for example).

Comment thread src/main/java/io/github/felipebonezi/cipherizy/algorithm/AESCipher.java Outdated
@felipebonezi felipebonezi self-assigned this Oct 21, 2022
@felipebonezi
felipebonezi force-pushed the feature/refactor-icipher branch from 10c0e14 to 9e78a15 Compare October 21, 2022 12:34
@felipebonezi
felipebonezi force-pushed the feature/refactor-icipher branch from 9e78a15 to 6589de4 Compare October 21, 2022 12:37
@felipebonezi
felipebonezi merged commit 431ac87 into main Oct 21, 2022
@felipebonezi
felipebonezi deleted the feature/refactor-icipher branch October 21, 2022 12:52
@felipebonezi felipebonezi added this to the 1.1.0 milestone Oct 21, 2022
@felipebonezi felipebonezi added type:feature Amazing new feature. tip:merge-when-ready Merge PR when is passing to all checks. labels Oct 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tip:merge-when-ready Merge PR when is passing to all checks. type:feature Amazing new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problems to encrypt and decrypt when uses encryptToString method

1 participant