search

cert-manager / website

Source code for the cert-manager.io website, including project documentation
https://cert-manager.io
Apache License 2.0
53 stars 333 forks source link

Venafi Issuer: Explain why "Do not select 'refresh token enabled'" #1373

Closed maelvls closed 9 months ago

maelvls commented 9 months ago

A while back, when the "Access Token" authentication method was documented by Richard in https://github.com/cert-manager/website/pull/346, the following was added:

NOTE: Do not select "Refresh Token Enabled" and set a long "Token Validity (days)".

I have read a few questions about "why is cert-manager not supporting refresh tokens?", so I figured I would explain why in this PR.

netlify[bot] commented 9 months ago

Deploy Preview for cert-manager-website ready!

Name Link
Latest commit 7307f0c9a12751ef39cedcb966c470c33603e334
Latest deploy log https://app.netlify.com/sites/cert-manager-website/deploys/658402d5d0e5650008601859
Deploy Preview https://deploy-preview-1373--cert-manager-website.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

jetstack-bot commented 9 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wallrj

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/cert-manager/website/blob/master/OWNERS)~~ [wallrj] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment