Modified final hint to point to the correct mapping (/conf instead of /config). Minor code syntax cleanup

git-svn-id: http://webgoat.googlecode.com/svn/trunk@110 4033779f-a91e-0410-96ef-6bf7bf53c507
This commit is contained in:
esheri3 2007-02-02 15:04:37 +00:00
parent e748aa0e90
commit dda693f0ff

View File

@ -53,8 +53,7 @@ public class ForcedBrowsing extends LessonAdapter
private final static String SUCCEEDED = "succeeded";
private final static IMG MAC_LOGO = new IMG("images/logos/macadamian.gif").setAlt(
"Macadamian Technologies").setBorder(0).setHspace(0).setVspace(0);
private final static IMG MAC_LOGO = new IMG("images/logos/macadamian.gif").setAlt("Macadamian Technologies").setBorder(0).setHspace(0).setVspace(0);
/**
* Description of the Method
@ -65,19 +64,16 @@ public class ForcedBrowsing extends LessonAdapter
protected Element createContent(WebSession s)
{
ElementContainer ec = new ElementContainer();
String success = new String(s.getParser().getStringParameter(SUCCEEDED,
""));
String success = new String(s.getParser().getStringParameter(SUCCEEDED,""));
if (success.length() != 0 && success.equals("yes"))
{
ec.addElement(new BR().addElement(new H1()
.addElement("Welcome to WebGoat Configuration Page")));
ec.addElement(new BR().addElement(new H1().addElement("Welcome to WebGoat Configuration Page")));
ec.addElement(new BR());
Table t1 = new Table().setCellSpacing(0).setCellPadding(0)
.setBorder(0).setWidth("90%").setAlign("center");
Table t1 = new Table().setCellSpacing(0).setCellPadding(0).setBorder(0).setWidth("90%").setAlign("center");
TR tr = new TR();
tr.addElement(new TD(
new StringElement("Set Admin Privileges for: ")));
tr.addElement(new TD(new StringElement("Set Admin Privileges for: ")));
Input input1 = new Input(Input.TEXT, "", "");
tr.addElement(new TD(input1));
@ -98,9 +94,7 @@ public class ForcedBrowsing extends LessonAdapter
}
else
{
ec
.addElement("Can you try to force browse to the config page which "
+ "should only be accessed by maintenance personnel.");
ec.addElement("Can you try to force browse to the config page which should only be accessed by maintenance personnel.");
}
return ec;
}
@ -113,7 +107,6 @@ public class ForcedBrowsing extends LessonAdapter
*/
protected Category getDefaultCategory()
{
return AbstractLesson.A10;
}
@ -128,9 +121,8 @@ public class ForcedBrowsing extends LessonAdapter
List<String> hints = new ArrayList<String>();
hints.add("Try to guess the URL for the config page");
hints.add("The config page is guessable and hackable");
hints
.add("Play with the URL and try to guess what the can you replace 'attack' with.");
hints.add("Try to navigate to http://localhost/WebGoat/config");
hints.add("Play with the URL and try to guess what the can you replace 'attack' with.");
hints.add("Try to navigate to http://localhost/WebGoat/conf");
return hints;
}