-
Notifications
You must be signed in to change notification settings - Fork 297
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
[DEVOPS-299] Fix TLS certificate path #454
Conversation
@domenkozar, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DominikGuzei, @darko-mijic and @tomothespian to be potential reviewers. |
@fersel can you test win installer if certs work (and if not, paste the beginning of the daedalus log)? |
@domenkozar certs are working correctly for the provided installer. |
@domenkozar not so lucky with the mac one, certs are not created :
The beginning of the daedalus log:
Tell me if you need full log(s). |
I think I know what's the problem. |
e4ad17f
to
252854a
Compare
@domenkozar that fixes the topology issue, thanks! The node seems to work OK now. But now the frontend part wouldn't start because of the |
89e4d87
to
1490b99
Compare
@domenkozar we're getting closer :) now I'm getting
From Daedalus.log:
(Note that certs are being "fetched by" the node though). |
1490b99
to
c29f077
Compare
Rebase of master broke things :) Should be fixed now. |
@domenkozar now this:
(node is ok but frontend is not working) |
c29f077
to
94ce34c
Compare
@fersel confirmed that it works. |
The code looks good to me. I have also verified that installer is now working properly on Mac @domenkozar. |
Testing...