Remove WebGoat session object by nbaars · Pull Request #1929 · WebGoat/WebGoat

added 9 commits

October 26, 2024 09:18
Always been a bit of an ugly construction, as none of the dependencies are clear. The constructors are hidden due to autowiring the base class. This PR removes two of the fields.

As a bonus we now wire the authentication principal directly in the controllers.
No more need to get the current lesson set in a session. The lesson is now passed to the endpoints.

@nbaars

Since we have Windows specific paths let's run it standalone for now. We need to run these tests on Docker as well (for now disabled)

@nbaars nbaars changed the title Use Testcontainer for testing WebGoat Remove WebGoat session object

Oct 26, 2024

@nbaars nbaars deleted the nbaars/test-containers branch

October 26, 2024 08:54