Improve variable combined cert vars about clank HOT 10 CLOSED

(R.I.P. Ratchet)

from clank.

Ahh, Okay I see what you mean now. Yeah the format i'd rather see here is:

COMBINED_CERT_FILE = self_signed_combined.crt
COMBINED_CERT_DIR = /etc/ssl/certs
COMBINED_CERT_PATH = "{{COMBINED_CERT_DIR}}/{{COMBINED_CERT_FILE}}"
...

Thanks for the clarification!

from clank.

I agree Connor. Thanks for noting this.

The nginx and uwsgi.ini sections of variables will be re-worked this coming sprint.

from clank.

I am adding this to the v2.1 milestone and accepting the issue as assigned to me.

from clank.

This effort will be related to #42.

from clank.

👍

from clank.

I think this is something that Steve might be working toward. Clearing my assignment.

from clank.

I believe this is no longer a problem -- @cdosborn can you confirm?

from clank.

This is what tripped me up when I was configuring:

COMBINED_CERT_FILE = self_signed_combined.crt
COMBINED_CERT_PATH = /etc/ssl/certs/self_signed_combined.crt
KEY_FILE = self-signed.key
KEY_PATH = /etc/ssl/private

The combined cert path includes the cert file. I'm not sure why we want both to be configurable.

from clank.

I would love it if we have a convention for ..._FILE, ..._DIR, ..._PATH.

from clank.