digininja / DVWA

Damn Vulnerable Web Application (DVWA)
GNU General Public License v3.0
10.36k stars 3.65k forks source link

feat: allow configuration via environment variables #592

Closed alles-klar closed 10 months ago

alles-klar commented 1 year ago

Adding the option to configure the dvwa in a docker container via environment variables. This allows users to change the config without rebuiling the docker container.

digininja commented 1 year ago

Looks good, could you write it up into the README so people know it exists and how to use it.

alles-klar commented 1 year ago

I was only able to modify the english version. Hope that helps.

digininja commented 1 year ago

That's fine, I'll have a look later.

I've asked all the foreign language translators to keep an eye on the README so they can keep it updated.

On Tue, 14 Nov 2023, 19:03 alles-klar, @.***> wrote:

I was only able to modify the english version. Hope that helps.

— Reply to this email directly, view it on GitHub https://github.com/digininja/DVWA/pull/592#issuecomment-1810980498, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA4SWOR7RSBD6G2W7WPKFLYEO57RAVCNFSM6AAAAAA7JVX4KGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMJQHE4DANBZHA . You are receiving this because you commented.Message ID: @.***>

digininja commented 10 months ago

Sorry, I missed merging this. Thanks for the contribution.