Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

After login, there is no default lesson #72

Closed
davewichers opened this issue Sep 15, 2015 · 5 comments
Closed

After login, there is no default lesson #72

davewichers opened this issue Sep 15, 2015 · 5 comments
Assignees
Milestone

Comments

@davewichers
Copy link

You kind of get a blank screen with just the menus on the side. Seems like you should default to go to the Http Basics lesson, which is what previous WebGoat used to do.

@misfir3
Copy link
Contributor

misfir3 commented Sep 15, 2015

Thanks for creating this officially. This has been on my mental //TODO list, but did not get it noted down here yet.

@misfir3 misfir3 self-assigned this Sep 15, 2015
@misfir3 misfir3 added this to the 7.0 Release milestone Sep 15, 2015
@mayhew64
Copy link
Contributor

Let's default it to "How to Work with WebGoat"

@misfir3
Copy link
Contributor

misfir3 commented Sep 17, 2015

@mayhew64 - I set up (not committed or PR'd yet) a ['well call it a]'page' that would show up at start.mvc#welcome (which you would now be redirected to on login). Since 'How to Work with Webgoat' is automatically 'completed' on viewing the page and the page would come up by default, I propose that we just move that content over to the template/page and remove 'How to Work with WebGoat' as a plugin lesson. Thoughts?

@mayhew64
Copy link
Contributor

I would prefer to have it default to the first lesson, i.e. lesson with the lowest rank. This way, the user can reconfigure what lesson is defaulted from the webgoat.properties. Even if there are no plugins installed, I put the admin screens in the webgoat-container, so the report card screen would be the default in that case.

dougmorato added a commit that referenced this issue Sep 18, 2015
#72, defaulting to firstLesson on initial redirect
@dougmorato
Copy link
Member

@misfir3 pushed code to redirect to first lesson upon login

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants