Skip to content

feat(challenge): add Challenge 63 hardcoded AES encryption key#2492

Merged
commjoen merged 11 commits intoOWASP:masterfrom
moeedrehman135:feat/challenge63-hardcoded-encryption-key
Apr 18, 2026
Merged

feat(challenge): add Challenge 63 hardcoded AES encryption key#2492
commjoen merged 11 commits intoOWASP:masterfrom
moeedrehman135:feat/challenge63-hardcoded-encryption-key

Conversation

@moeedrehman135
Copy link
Copy Markdown
Contributor

What kind of changes does this PR include?

  • Fixes or refactors
  • A new challenge
  • Additional documentation
  • Something else

Description

Adds Challenge 63 demonstrating bad encryption practices — hardcoding an AES encryption key and IV directly in source code alongside the ciphertext, making the encryption completely ineffective.

Changes:

  • Challenge63.java — AES-CBC decryption with hardcoded key/IV and ciphertext
  • challenge63.adoc — challenge description
  • challenge63_hint.adoc — step-by-step hint for players
  • challenge63_reason.adoc — explanation of why this is dangerous
  • Registered in wrong-secrets-configuration.yaml under crypto category
  • 3 unit tests passing

Testing:

  • Correct answer accepted
  • Wrong answer rejected
  • Spoiler reveals correct answer

Closes #297

Adds a new challenge demonstrating bad encryption practices where
a secret is encrypted with AES-CBC but the key and IV are hardcoded
in the same source file, making the encryption ineffective.
- Add Challenge63.java with hardcoded AES key/IV and ciphertext
- Add explanation, hint, and reason adoc files
- Register in wrong-secrets-configuration.yaml under crypto category
- Add unit tests (3 passing)
Closes OWASP#297
Comment thread src/main/java/org/owasp/wrongsecrets/challenges/docker/challenge63/Challenge63.java Dismissed
@commjoen
Copy link
Copy Markdown
Collaborator

Hi @moeedrehman135 ! looking great! Hope to review in depth after the weekend.

Copy link
Copy Markdown
Collaborator

@commjoen commjoen left a comment

Choose a reason for hiding this comment

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

Hey @moeedrehman135 , thank you for your awesome work! This looks great!
can you please fix the feedback, so we can merge?

Comment thread src/main/resources/explanations/challenge63.adoc Outdated
Comment thread src/main/resources/explanations/challenge63_hint.adoc Outdated
Comment thread src/main/resources/explanations/challenge63_hint.adoc
Comment thread src/test/java/org/owasp/wrongsecrets/challenges/docker/Challenge63Test.java Outdated
Comment thread src/test/java/org/owasp/wrongsecrets/challenges/docker/Challenge63Test.java Outdated
@commjoen
Copy link
Copy Markdown
Collaborator

Thank you for your awesome new challenge @moeedrehman135 ! we will fix the remaining issues after merge :)

@commjoen commjoen merged commit 81d9e73 into OWASP:master Apr 18, 2026
12 of 20 checks passed
@commjoen commjoen self-requested a review April 18, 2026 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add hardcoded encryption key on top of a secret.

3 participants