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.

Sign up for GitHub

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

Jump to bottom

Path for tls credentials #2521

Closed
AuspeXeu opened this issue Oct 18, 2018 · 3 comments · Fixed by haraka/haraka-config#46
Closed

Path for tls credentials #2521

AuspeXeu opened this issue Oct 18, 2018 · 3 comments · Fixed by haraka/haraka-config#46
Labels
Feature Request

Comments

@AuspeXeu
Copy link
Contributor

It seems like the paths given for cert and key option of the tls plugin are always relative to the config folder. This is not very good if you want to separate the configuration files from the credentials. I think absolute paths should be supported.
@msimerson
Copy link
Member

moved to wiki

msimerson added a commit to haraka/haraka-config that referenced this issue Dec 20, 2018
@msimerson
permit retrieval of fully qualified path
a43aaae 
fixes haraka/Haraka#2521
@msimerson msimerson mentioned this issue Dec 20, 2018
Merged
@msimerson
Copy link
Member

Update. See haraka/haraka-config#46

msimerson added a commit to haraka/haraka-config that referenced this issue Dec 20, 2018
@msimerson
refactor config.js as es6 class (#46)
4908010 
* refactor config.js as es6 class
* use path.resolve instead of ./dir/file
* test: replace a few functions with fat arrows
* watch: recursive=true, fixes haraka/Haraka#2566
* permit retrieval of fully qualified path, fixes haraka/Haraka#2521
@msimerson
Copy link
Member

Completed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

refactor config.js as es6 class haraka/haraka-config
2 participants
@msimerson @AuspeXeu