Skip to content

Commit cc623d8

Browse files
committed
fix: vault template injection challenge renamed to 46
1 parent dd1d629 commit cc623d8

File tree

9 files changed

+16
-16
lines changed

9 files changed

+16
-16
lines changed

k8s/secret-challenge-vault-deployment.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ spec:
2525
vault.hashicorp.com/agent-inject-status: "update"
2626
vault.hashicorp.com/namespace: "default"
2727
vault.hashicorp.com/log-level: debug
28-
vault.hashicorp.com/agent-inject-secret-challenge44: "secret/data/secret-challenge"
29-
vault.hashicorp.com/agent-inject-template-challenge44: |
28+
vault.hashicorp.com/agent-inject-secret-challenge46: "secret/data/secret-challenge"
29+
vault.hashicorp.com/agent-inject-template-challenge46: |
3030
{{ with secret "/secret/data/secret-challenge" }}
3131
{{ range $k, $v := .Data.data }}
3232
{{ printf "%s=%s" $k $v }}

src/main/java/org/owasp/wrongsecrets/challenges/kubernetes/Challenge44.java renamed to src/main/java/org/owasp/wrongsecrets/challenges/kubernetes/Challenge46.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,12 @@
77

88
/** This challenge is about having a secrets injected via Vault template. */
99
@Component
10-
public class Challenge44 extends FixedAnswerChallenge {
10+
public class Challenge46 extends FixedAnswerChallenge {
1111

1212
private final Vaultpassword vaultPassword;
1313
private final String mockedAnswer;
1414

15-
public Challenge44(Vaultpassword vaultPassword, @Value("${vaultpassword}") String mockedAnswer) {
15+
public Challenge46(Vaultpassword vaultPassword, @Value("${vaultpassword}") String mockedAnswer) {
1616
this.vaultPassword = vaultPassword;
1717
this.mockedAnswer = mockedAnswer;
1818
}
File renamed without changes.
File renamed without changes.
File renamed without changes.

src/main/resources/wrong-secrets-configuration.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -711,13 +711,13 @@ configurations:
711711
ctf:
712712
enabled: true
713713

714-
- name: Challenge 44
715-
short-name: "challenge-44"
714+
- name: Challenge 46
715+
short-name: "challenge-46"
716716
sources:
717-
- class-name: "org.owasp.wrongsecrets.challenges.kubernetes.Challenge44"
718-
explanation: "explanations/challenge44.adoc"
719-
hint: "explanations/challenge44_hint.adoc"
720-
reason: "explanations/challenge44_reason.adoc"
717+
- class-name: "org.owasp.wrongsecrets.challenges.kubernetes.Challenge46"
718+
explanation: "explanations/challenge46.adoc"
719+
hint: "explanations/challenge46_hint.adoc"
720+
reason: "explanations/challenge46_reason.adoc"
721721
environments: [*k8s_vault]
722722
difficulty: *expert
723723
category: *vault

src/test/java/org/owasp/wrongsecrets/challenges/cloud/ChallengesControllerWithPresetCloudValuesTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ void shouldSpoilExercises() throws Exception {
5353
void shouldNotShowDisabledChallengeAnywhere() throws Exception {
5454
for (var challenge : challenges.getChallengeDefinitions()) {
5555
var shortname = challenge.name().shortName();
56-
if (shortname.contains("44")) {
56+
if (shortname.contains("46")) {
5757
continue;
5858
}
5959
mvc.perform(get("/challenge/%s".formatted(challenge.name().shortName())))

src/test/java/org/owasp/wrongsecrets/challenges/kubernetes/Challenge44Test.java renamed to src/test/java/org/owasp/wrongsecrets/challenges/kubernetes/Challenge46Test.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,13 @@
44

55
import org.junit.jupiter.api.Test;
66

7-
public class Challenge44Test {
7+
public class Challenge46Test {
88

99
@Test
1010
void spoilerShouldGiveAnswerWithVault() {
1111
var vaultPassword = new Vaultpassword();
1212
vaultPassword.setPassword("answer");
13-
var challenge = new Challenge44(vaultPassword, "");
13+
var challenge = new Challenge46(vaultPassword, "");
1414
assertThat(challenge.spoiler().solution()).isNotEmpty();
1515
assertThat(challenge.answerCorrect(challenge.spoiler().solution())).isTrue();
1616
}
@@ -19,7 +19,7 @@ void spoilerShouldGiveAnswerWithVault() {
1919
void spoilerShouldGiveAnswer() {
2020
var vaultPassword = new Vaultpassword();
2121
vaultPassword.setPassword("");
22-
var challenge = new Challenge44(vaultPassword, "answer");
22+
var challenge = new Challenge46(vaultPassword, "answer");
2323
assertThat(challenge.spoiler().solution()).isEqualTo("answer");
2424
assertThat(challenge.answerCorrect(challenge.spoiler().solution())).isTrue();
2525
}
@@ -28,7 +28,7 @@ void spoilerShouldGiveAnswer() {
2828
void incorrectAnswerShouldNotSolveChallenge() {
2929
var vaultPassword = new Vaultpassword();
3030
vaultPassword.setPassword("answer");
31-
var challenge = new Challenge44(vaultPassword, "");
31+
var challenge = new Challenge46(vaultPassword, "");
3232
assertThat(challenge.answerCorrect("wrong answer")).isFalse();
3333
}
3434
}

src/test/java/org/owasp/wrongsecrets/challenges/kubernetes/ChallengesControllerWithPresetKubernetesValuesTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ void shouldNotShowDisabledChallengeAnywhere() throws Exception {
5454
|| shortname.contains("9")
5555
|| shortname.contains("10")
5656
|| shortname.contains("11")
57-
|| shortname.contains("44")) {
57+
|| shortname.contains("46")) {
5858
continue;
5959
}
6060
mvc.perform(get("/challenge/%s".formatted(challenge.name().shortName())))

0 commit comments

Comments
 (0)