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

TLS for slave HTTP #32

Closed
7 tasks done
janisstreib opened this issue Aug 24, 2016 · 3 comments
Closed
7 tasks done

TLS for slave HTTP #32

janisstreib opened this issue Aug 24, 2016 · 3 comments
Assignees

Comments

@janisstreib
Copy link
Member

janisstreib commented Aug 24, 2016

I prupose the follwing procedure:

  • Each slave has its (not necessarily unique) cert and private key, signed by a cluster local CA and supplied via the cmd.
  • When the mater connects to the slave, it only allows connections with certs signed by the CA.
  • The CA/PKI structure will not be managed by the master.

=> Therefore I prupose to place the CA cert into the master as reference to simplify the implementation.

  • Add cert/key parameters to the slave cmd
  • Serve http with TLS instead of plain HTTP on the slave
    • Only use proper ciphers, please ;)
  • Use http with TLS + client cert instead of plain HTTP on the master
  • Configure the CA in the master
    • (Optional) Add CA to MaterAPI (/api/system)
    • (Optional) Add GUI setting or display for the CA
    • Alternatively: Add commandline parameter for the reference CA to master cmd
  • Implement connection check in the deployer/MSP and generate appropriate probelms
  • Implement client cert check in the slave
  • Document procedure in the manual (see Add user manual/install instructions #33)
@problame
Copy link
Member

remove hardcoded endpoints

@janisstreib
Copy link
Member Author

janisstreib commented Aug 31, 2016

Slave verification by master done with 444519b

@janisstreib
Copy link
Member Author

Implemented cert auth in 837f43b

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