Liblor / applied_sec_lab

Applied Security Laboratory - AS19
6 stars 1 forks source link

Configure HAproxy load balancing, implement basic health checks #62

Closed RequestForCoffee closed 4 years ago

RequestForCoffee commented 4 years ago

Relates to #35.

Summary of changes:

Remaining work:

  1. Test (and troubleshoot as necessary) a two-chain setup with duplicate boxes
  2. Configure a multi-peer HAproxy solution to synchronize session stick tables across aslld01 and aslld02+ so that in the event of a load balancer going down, the associated sessions are not interrupted
RequestForCoffee commented 4 years ago

https://hynek.me/articles/hardening-your-web-servers-ssl-ciphers/ explains that the ciphersuites is apparently already "hardened".

Miro-H commented 4 years ago

https://hynek.me/articles/hardening-your-web-servers-ssl-ciphers/ explains that the ciphersuites is apparently already "hardened".

Yeah, but he focuses quite much on backwards compatibility. Additionally, despite the authors promise to update his blog regularly, the last one was over a year ago. The ciphers on the very bottom, in the section "I only care about web browsers released in the past five years." is more what I'd imagined:

global
   ssl-default-bind-options force-tlsv12 prefer-client-ciphers
   ssl-default-bind-ciphers ECDH+AESGCM:ECDH+CHACHA20:ECDH+AES256:ECDH+AES128:!aNULL:!SHA1:!AESCCM

   ssl-default-server-options force-tlsv12
   ssl-default-server-ciphers ECDH+AESGCM:ECDH+CHACHA20:ECDH+AES256:ECDH+AES128:!aNULL:!SHA1:!AESCCM
keyctl commented 4 years ago

I merged this with the current master, but it gives me a PR_END_OF_FILE_ERROR. @RequestForCoffee Have you experienced this before?

keyctl commented 4 years ago

Okay, I think I made a mistake during merge. Somehow the CrlDistributionPoint was renamed somewhere. @RequestForCoffee Please confirm, that the current branch is semantically as it should be. a40712d resolved my issue described before.

RequestForCoffee commented 4 years ago

@keyctl Yes, a40712d is semantically correct - CrlDistributionPoint is now constant (under imovies.ch) and no longer a format string. Should be good to merge?

RequestForCoffee commented 4 years ago

@Miro-H do you want to take a look or should I merge this?