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.

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

Update to add required formatting for X.509 certificate #6939

Open
wants to merge 1 commit into
base: current
Choose a base branch
from

Conversation

Tonayya
Copy link
Contributor

@Tonayya Tonayya commented Feb 21, 2025

What are you changing in this pull request and why?

We have not mentioned the necessary formatting required for the X.509 certificate that users are required to upload and there are varying values of this available, but we only support the PEM formatting. So just added that information in the sections where this certificate was mentioned. We can also add an example of how this should look but I wasnt sure where to put it. Below is what it should look like (feel free to share this, it's expired/invalid/safe to use):

-----BEGIN CERTIFICATE-----
MIIC8DCCAdigAwIBAgIQNOg8W0wdxZxD21/sKwxA3DANBgkqhkiG9w0BAQsFADA0MTIwMAYDVQQD
EylNaWNyb3NvZnQgQXp1cmUgRmVkZXJhdGVkIFNTTyBDZXJ0aWZpY2F0ZTAeFw0yMzEyMjIwMDU1
MDNaFw0yNjEyMjIwMDU1MDNaMDQxMjAwBgNVBAMTKU1pY3Jvc29mdCBBenVyZSBGZWRlcmF0ZWQg
U1NPIENlcnRpZmljYXRlMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAqfXQGc/D8ofK
aXbPXftPotqYLEQtvqMymgvhFuUm+bQ9YSpS1zwNQ9D9hWVmcqis6gO/VFw61e0lFnsOuyx+XMKL
rJjAIsuWORavFqzKFnAz7hsPrDw5lkNZaO4T7tKs+E8N/Qm4kUp5omZv/UjRxN0XaD+o5iJJKPSZ
PBUDo22m+306DE6ZE8wqxT4jTq4g0uXEitD2ZyKa6Wo8bZyMW9tKl5oiCB47Pgn/mpqae9o0Q2aQ
LP9zosNZ07IjKkIfyFKMP7xHwzrl5a60y0rSIYS/edqwEhkpzaz0f8QW5pws668CpZ1AVgfP9TtD
Y1EuxBSDQoY5TLR8++2eH4te0QIDAQABMA0GCSqGSIb3DQEBCwUAA4IBAQCEts9ujwaokRGfdtgH
76kGrRHiFVWTyWdcpl1dNDvGhUtCRsTC76qwvCcPnXBebVi54Nf0ik4oSwwQJALExriSvxtcNW1b
qvnY52duXeZ1CSfwHkHkQLyWBANv8ZCkgtcSWnoTafOJAJPUo4aSrAAY2s5hP3ukWdV9zQscUw2b
GwN0/bTxxQgA2NLZzFuHSnkuRX56xRKhVu21USPTHMGmJYBqZqwePZXTcyxp64f3Mtj3g327r/qZ
squyPSq5BrF4ivguYoTcGg4SCP7qfiNRFyypY3UT9ZUN0n46MuPmVC7vXTsPRQtNRTpJj/b2gGLk
1RcPb1JosS1ct5Mtjs41
-----END CERTIFICATE-----

Checklist

  • I have reviewed the Content style guide so my content adheres to these guidelines.
  • The topic I'm writing about is for specific dbt version(s) and I have versioned it according to the version a whole page and/or version a block of content guidelines.
  • I have added checklist item(s) to this list for anything anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."
  • The content in this PR requires a dbt release note, so I added one to the release notes page.

🚀 Deployment available! Here are the direct links to the updated files:

@Tonayya Tonayya requested a review from a team as a code owner February 21, 2025 22:08
Copy link

vercel bot commented Feb 21, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Feb 21, 2025 10:08pm

@github-actions github-actions bot added content Improvements or additions to content size: x-small This change will take under 3 hours to fix. labels Feb 21, 2025
@@ -87,7 +87,7 @@ them to complete setup in dbt Cloud.

- Identity Provider Issuer
- Identity Provider SSO Url
- X.509 Certificate
- X.509 Certificate (PEM format required)
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [vale] reported by reviewdog 🐶
[custom.Typos] Oops there's a typo -- did you really mean 'X.509'?

@@ -421,7 +421,7 @@

- Issuer URL
- SAML 2.0 Endpoint (HTTP)
- X.509 Certificate
- X.509 Certificate (PEM format required)
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [vale] reported by reviewdog 🐶
[custom.Typos] Oops there's a typo -- did you really mean 'X.509'?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content size: x-small This change will take under 3 hours to fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant