learning-unlimited/ESP-Website — comment on commit 5ed0b71832c68b617eb7c19c870249f2ce7a4370
File:  line 0
By gkanwar:

Ah I see. I thought you meant that having the Varnish server with our server pointing to it could open a security flaw to the Django server itself. This makes more sense.

I think I would prefer to fix the test suite so that it doesn't crash without these set (I can work on this). This way, as you said, someone without Varnish set up can still have tests work. I think if we make this change, it makes more sense to move this setting into local_settings, because now it isn't required to make things work, and seems like something that should be modified (and also possibly not exist) on a case-by-case basis.