Skip to content
This repository was archived by the owner on Dec 8, 2022. It is now read-only.

Bugfix/improve encryption #51

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

nitinblotout
Copy link
Contributor

Instead of AES algorithm we are using "AES/CBC/PKCS5Padding" algorithm.

@nitinblotout nitinblotout self-assigned this Apr 2, 2022
@github-actions
Copy link

github-actions bot commented Apr 2, 2022

📚 Deployed Docs to https://docs-android-qzothkc68-nejczdovc.vercel.app

@nitinblotout nitinblotout linked an issue Apr 2, 2022 that may be closed by this pull request
@@ -115,7 +115,7 @@ class EncryptionUtils(algorithm: String="", passphrase: String="", mode: Int=0)

private fun encrypt(raw: ByteArray, clear: ByteArray): ByteArray {
return try {
val skeySpec = SecretKeySpec(raw, "AES")
val skeySpec = SecretKeySpec(raw, ALGORITHM_AES_CBC_PKCS5Padding)
Copy link
Contributor

Choose a reason for hiding this comment

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

@NejcZdovc NejcZdovc self-requested a review July 14, 2022 07:36
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.

Analysis Security with veracode tool
2 participants