Skip to content
This repository has been archived by the owner on Apr 17, 2024. It is now read-only.

docs: add isntructions for installing KDCC on demo #47

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

zalsader
Copy link
Contributor

@zalsader zalsader commented Oct 5, 2023

No description provided.

@linear
Copy link

linear bot commented Oct 5, 2023

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 5, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 52caafd
Status:⚡️  Build in progress...

View logs

Copy link
Contributor

@ideepika ideepika left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me! thanks for documenting!

# The ACME server URL
server: https://acme-staging-v02.api.letsencrypt.org/directory
# Email address used for ACME registration
email: [email protected]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make it the admin email instead of your company email:

Suggested change
email: zuhair@koor.tech
email: admin@koor.tech

spec:
acme:
# The ACME server URL
server: https://acme-staging-v02.api.letsencrypt.org/directory
Copy link
Member

@galexrt galexrt Oct 5, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~~You are using the Letsencrypt Staging Server. These certificates aren't "valid" without manually installing the staging CA (which you shouldn't).

Please switch to the "live"/production Letsencrpyt servers to make sure we have valid certificates for everyone.~~

Edit: I am not sure why you create the staging issuer if the prod one is used anyway here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was used for testing the system at the beginning since the prod is rate-limited. I removed it form the docs now.

@zalsader zalsader merged commit 8d277e2 into main Oct 5, 2023
1 of 2 checks passed
@zalsader zalsader deleted the feature/gen-249 branch October 5, 2023 16:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants