dubey/WebGoat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: reset form and quiz color on reset lesson (#1903)

Browse Source 
* ./mvnw spotless:apply

```
[INFO] --- spotless-maven-plugin:2.41.1:apply (default-cli) @ webgoat ---
[INFO] Writing clean file: /home/ulyssa/labs/WebGoat/WebGoat-bb6e84d/src/main/java/org/owasp/webgoat/lessons/sqlinjection/introduction/SqlInjectionLesson5a.java
```

* On reset lesson: reset form and quizzes colors
This commit is contained in:
François Capon
2024-10-26 09:22:18 +02:00
committed by GitHub
parent f4c86be6c7
commit cb7c508046
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/org/owasp/webgoat/lessons/sqlinjection/introduction/SqlInjectionLesson5a.java
View File

@ -41,8 +41,8 @@ public class SqlInjectionLesson5a extends AssignmentEndpoint {
+ " '1'</span> always evaluates to true (The string ending literal for '1 is closed by" + " '1'</span> always evaluates to true (The string ending literal for '1 is closed by"
+ " the query itself, so you should not inject it). So the injected query basically looks" + " the query itself, so you should not inject it). So the injected query basically looks"
+ " like this: <span style=\"font-style: italic\">SELECT * FROM user_data WHERE" + " like this: <span style=\"font-style: italic\">SELECT * FROM user_data WHERE"
+ " first_name = 'John' and last_name = '' or TRUE</span>, which will always evaluate to" + " (first_name = 'John' and last_name = '') or (TRUE)</span>, which will always evaluate"
+ " true, no matter what came before it."; + " to true, no matter what came before it.";
private final LessonDataSource dataSource; private final LessonDataSource dataSource;
public SqlInjectionLesson5a(LessonDataSource dataSource) { public SqlInjectionLesson5a(LessonDataSource dataSource) {

3
src/main/resources/webgoat/static/js/goatApp/view/LessonContentView.js
View File

@ -147,7 +147,6 @@ define(['jquery',
}, },
markAssignmentComplete: function () { markAssignmentComplete: function () {
this.curForm.reset();
$(this.curForm).siblings('.assignment-success').find('i').removeClass('hidden'); $(this.curForm).siblings('.assignment-success').find('i').removeClass('hidden');
this.paginationControlView.updateCollection(); this.paginationControlView.updateCollection();
}, },
@ -222,6 +221,8 @@ define(['jquery',
}, },
resetLesson: function () { resetLesson: function () {
this.$el.find(form).trigger("reset");
this.$el.find(".quiz_question").removeClass('correct incorrect');
this.$el.find('.attack-feedback').hide(); this.$el.find('.attack-feedback').hide();
this.$el.find('.attack-output').hide(); this.$el.find('.attack-output').hide();
this.markAssignmentIncomplete(); this.markAssignmentIncomplete();