Skip to content

Conversation

@addanus
Copy link

@addanus addanus commented Jul 10, 2021

This how-to has 10 steps and sample files. I wasn't sure where/how exactly to contribute it so just added a link.

@maebeam / @lazynina - thoughts?

Setting up SSL - gcp-nginx-docker

This how-to has 10 steps and sample files. I wasn't sure where/how exactly to contribute it so just added a link. 

@maebeam / @lazynina - thoughts?

[Setting up SSL - gcp-nginx-docker](https://github.com/addanus/bitclout_helper/tree/main/SSL-support)
@addanus addanus requested a review from a team as a code owner July 10, 2021 01:03
@maebeam
Copy link
Contributor

maebeam commented Jul 10, 2021

I would love for this information to have a more permanent home at

https://docs.bitclout.com/devs/running-a-node#how-do-i-configure-ssl

@addanus
Copy link
Author

addanus commented Jul 10, 2021

Me too. Can someone add the sample files? I don't have access to do that

@maebeam
Copy link
Contributor

maebeam commented Jul 10, 2021

I'm not quite sure what you mean. You should be able to open a pull request on the docs repo and add whatever you need

@addanus
Copy link
Author

addanus commented Jul 10, 2021

Got ya. 1 sec

@addanus
Copy link
Author

addanus commented Jul 10, 2021

Moving to docs

@addanus addanus closed this Jul 10, 2021
@addanus
Copy link
Author

addanus commented Jul 10, 2021

...

@addanus addanus reopened this Jul 10, 2021
@addanus
Copy link
Author

addanus commented Jul 10, 2021

@maebeam all clear now - needed to use a different approach, since folder uploads on the site required more permissions. Single file creation worked, but ultimately vscode did the trick.

Current status:

  1. added PR on docs with the folder and its how-to readme.md, plus link out from top level readme
  2. this Get SSL working for a node #69 edited to point back to docs

Good to go for an approval if you concur

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants