Documentation fix

[Pradyumn-cloud]

Fix - #3244

Changes done as Mentioned in issue.

insecureSkipVerify set to true

added jwksUri: http://dex.auth.svc.cluster.local:5556/dex/keys

✅ Contributor Checklist

• I have tested these changes with kustomize. See Installation Prerequisites.
• All commits are signed-off to satisfy the DCO check.

[google-oss-prow]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign yanniszark for approval. For more information see the Kubernetes Code Review Process.

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

Needs approval from an approver in each of these files:

• common/dex/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[juliusvonkohout]

You need to sign you commits as desribed in https://github.com/kubeflow/manifests/pull/3257/checks?check_run_id=52443746965

[juliusvonkohout]

1. the default should be false
2. Please link in an inline comment about self-signed certificates to the dex documentation where this is documented

[juliusvonkohout]

Please revert this. It should be a comment for the variable in line 156 DEX_ISSUER="https://kubeflow.example.com/dex". Please also link to the dex documentation or something that explains the change.

[juliusvonkohout]

@Pradyumn-cloud your commits still have to be signed.

[juliusvonkohout]

I do not see "insecureSkipVerify" on the page you linked

[juliusvonkohout]

and an inline comment would be insecureSkipVerify: false # Set to true only for self-signed certificates. See: https://dexidp.io/docs/connectors/oidc/ in a single line

[juliusvonkohout]

insecureSkipVerify: false # Specify whether to communicate with the authentication provider without validating SSL certificates see https://dexidp.io/docs/connectors/oauth/ and https://dexidp.io/docs/connectors/oidc/

would be my guess but on the pages linked i see it in the oauth instead of the oidc section. CC @kromanow94 @kimwnasptd for more input

[Pradyumn-cloud]

oidc doc dont mention insecureSkipVerify instead oauth2.0 do.

So i am adding this line - insecureSkipVerify: false # Specify whether to communicate with the authentication provider without validating SSL certificates see https://dexidp.io/docs/connectors/oauth/ and https://dexidp.io/docs/connectors/oidc/.