Skip to content

Commit

Permalink
Updating Documenation (#10)
Browse files Browse the repository at this point in the history
Signed-off-by: Florian Wagner <florian.wagner@microsoft.com>
  • Loading branch information
Florian Wagner committed Dec 20, 2019
1 parent de1340a commit b90148f
Show file tree
Hide file tree
Showing 5 changed files with 126 additions and 185 deletions.
Binary file added Doc/img/architecture.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
59 changes: 0 additions & 59 deletions GeoBot/GeoBot.Tests/README.md

This file was deleted.

96 changes: 0 additions & 96 deletions GeoBot/GeoBot/README.md

This file was deleted.

16 changes: 15 additions & 1 deletion LetsEncrypt/Archive/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,18 @@ I tested this approach and while it is working fine (also for Custom DNS Name if
- Bad Hack to get the file back to the client (but maybe interesting in some ACI troubleshooting hacks, that is why I save this work here)
- Overall maybe a not so ideal solution from a security perspective to save the SSL certificate locally and chain it back to the Terraform script.

I will come up with a better more reproducible solution.
I will come up with a better more reproducible solution. But until then:

```bash
cd LetsEncrypt\Archive

az group create -name rg-geobot-global -location japaneast

.\IssueSSLCertificate.ps1 -YOUR_CERTIFICATE_EMAIL <YOUR_EMAIL> -YOUR_DOMAIN <BOT_NAME>.trafficmanager.net -BOT_NAME <BOT_NAME> -PFX_EXPORT_PASSWORD <PFX_EXPORT_PASSWORD>
```

Known issues/drawbacks/things to be reminded of for this script:

- __the BotName has to be unique__ since several Azure services will use it as prefix. Stick to lowercase no dashes and special chars and less than 20char. e.g. **myfirstname1234**
- due to a bug in the current setup a complex password with special characters **may not work** as expected
- the Terraform script in [Step 3](#3-deploy-the-solution) was created before realizing the need of creating a trusted SSL certificate, we will already deploy the TrafficManager in this step. In order to match with the default values of the current Terraform script, the resource group `rg-geobot-global` and location `japaneast` are correct. If you modify these values of the Terraform script either delete it (the resource group) after Step 2 before running Step 3 or match the values accordingly
Loading

0 comments on commit b90148f

Please sign in to comment.