From decc4262675da73fa708d44fdb52389c0ad93a66 Mon Sep 17 00:00:00 2001
From: "wirth.marcel" <wirth.marcel@4033779f-a91e-0410-96ef-6bf7bf53c507>
Date: Fri, 18 Apr 2008 08:38:51 +0000
Subject: [PATCH] Minor changes... Tan gets now only updatet after it was
 correct

git-svn-id: http://webgoat.googlecode.com/svn/trunk/webgoat@341 4033779f-a91e-0410-96ef-6bf7bf53c507
---
 .../org/owasp/webgoat/lessons/InsecureLogin.java         | 9 +++++----
 .../org/owasp/webgoat/lessons/MultiLevelLogin1.java      | 2 +-
 .../org/owasp/webgoat/lessons/MultiLevelLogin2.java      | 3 +--
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/main/project/JavaSource/org/owasp/webgoat/lessons/InsecureLogin.java b/main/project/JavaSource/org/owasp/webgoat/lessons/InsecureLogin.java
index ecf6e63c8..068858d9c 100644
--- a/main/project/JavaSource/org/owasp/webgoat/lessons/InsecureLogin.java
+++ b/main/project/JavaSource/org/owasp/webgoat/lessons/InsecureLogin.java
@@ -416,20 +416,21 @@ public class InsecureLogin extends SequentialLessonAdapter
 		Option httpOption = new Option();
 		httpOption.addElement("HTTP");
 		Option tcpOption = new Option();
-		tcpOption.addElement("TCP");
+		tcpOption.addElement("UDP");
+		Option ipsecOption = new Option();
+		ipsecOption.addElement("IPSEC");
 		Option msnmsOption = new Option();
 		msnmsOption.addElement("MSNMS");
 		Option tlsOption = new Option();
 		tlsOption.addElement("TLS");
 		protocolSelect.addElement(httpOption);
+		protocolSelect.addElement(ipsecOption);
 		protocolSelect.addElement(msnmsOption);
 		protocolSelect.addElement(tcpOption);
 		protocolSelect.addElement(tlsOption);
 		td4.addElement(protocolSelect);
-		//div.addElement(new BR());
 
-	
-		//div.addElement(new BR());
+
 		div.addElement(selectTable);
 		
 		div.addElement(new Input(Input.SUBMIT, "Submit", "Submit"));		
diff --git a/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin1.java b/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin1.java
index 9379a5ef0..537caac96 100644
--- a/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin1.java
+++ b/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin1.java
@@ -380,7 +380,6 @@ public class MultiLevelLogin1 extends SequentialLessonAdapter
 		ec.addElement(loginDiv);
 		ec.addElement(createLogoutLink());
 
-		updateTan(getLoggedInUser(s), s);
 	}
 
 	/**
@@ -409,6 +408,7 @@ public class MultiLevelLogin1 extends SequentialLessonAdapter
 	private void createSuccessfulLoginContent(WebSession s, ElementContainer ec)
 	{
 
+		updateTan(getLoggedInUser(s), s);
 		String userDataStyle = "margin-top:50px;";
 
 		Div userDataDiv = new Div();
diff --git a/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin2.java b/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin2.java
index 138a65ffb..d677844cd 100644
--- a/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin2.java
+++ b/main/project/JavaSource/org/owasp/webgoat/lessons/MultiLevelLogin2.java
@@ -383,7 +383,6 @@ public class MultiLevelLogin2 extends LessonAdapter
 		ec.addElement(loginDiv);
 		ec.addElement(createLogoutLink());
 
-		updateTan(user, s);
 	}
 
 	/**
@@ -404,7 +403,7 @@ public class MultiLevelLogin2 extends LessonAdapter
 
 	private void createSuccessfulLoginContent(WebSession s, ElementContainer ec, String user)
 	{
-
+		updateTan(user, s);
 		String userDataStyle = "margin-top:50px;";
 
 		Div userDataDiv = new Div();