feat: set resource revision for connectors #3868
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Nicholas Wiersma <nick@wiersma.co.za>
nrwiersma
force-pushed
the
connector-resource-revisions
branch
from
86eebc6 to
8405be6
Compare
|
I didn't find how it fixes the original problem, but still think that the change is ok. |
The issue is that updates to a Connector in the API are ignored because the ResourceVersion, which is empty, does not change, and therefore needs a restart of Dex to get the updated Connector. By adding and incrementing the ResourceVersion field, this solves the observed issue. |
nabokihms
approved these changes
Dec 17, 2024
|
Hey there @sagikazarmark @nabokihms, can you let us know when we can expect the next Dex release? |
xtremerui
pushed a commit
to concourse/dex
that referenced
this pull request
Feb 19, 2025
<!-- Release notes generated using configuration in .github/release.yml at v2.42.0 --> ## What's Changed ### Exciting New Features 🎉 * feat: also allow localhost equivalent IP addresses by @dsonck92 in dexidp#3778 ### Enhancements 🚀 * Added Discovery to grpc by @koendelaat in dexidp#3598 * feat(metrics): add response_size, request_duration histograms by @IvoGoman in dexidp#3748 * Handle root path better (than nothing) by @nabokihms in dexidp#3747 * Support base64 encoded and PEM encoded certs by @nabokihms in dexidp#3751 * GitLab connector: add GitLab additional group with role by @zvlb in dexidp#2941 * OIDC Connector: Support for IssuerAlias and group claims with maps instead of strings by @meldsza in dexidp#3676 * Add CSS for example app to make it prettier by @nabokihms in dexidp#3886 * feat: set resource revision for connectors by @nrwiersma in dexidp#3868 * Add authproxy preferred_username header by @kotx in dexidp#3950 * Passing context storage by @bobmaertz in dexidp#3941 ### Bug Fixes 🐛 * Fix scheme for DialURL ldap connection by @nabokihms in dexidp#3677 * Change workdir for gomplate by @nabokihms in dexidp#3684 * fix unmarshaling of expanded environment variables with special characters by @tuminoid in dexidp#3770 * saml connector: fix nil pointer PANIC on validate saml by @siarhei-haurylau in dexidp#3793 * fix: update enhancement proposal link by @emmanuel-ferdman in dexidp#3755 * Create offline sessions if approval is skipped by @nabokihms in dexidp#3828 ### Dependency Updates ⬆️ * build(deps): bump golang from 1.22.5-alpine3.20 to 1.23.1-alpine3.20 by @dependabot in dexidp#3728 * build(deps): bump alpine from 3.20.2 to 3.20.3 by @dependabot in dexidp#3729 * build(deps): bump golang.org/x/net from 0.27.0 to 0.29.0 by @dependabot in dexidp#3726 * build(deps): bump golang.org/x/oauth2 from 0.21.0 to 0.23.0 in /examples by @dependabot in dexidp#3722 * build(deps): bump actions/attest-build-provenance from 1.4.0 to 1.4.3 by @dependabot in dexidp#3727 * build(deps): bump google.golang.org/grpc from 1.65.0 to 1.66.1 in /examples by @dependabot in dexidp#3731 * build(deps): bump distroless/static-debian12 from `8dd8d3c` to `42d15c6` by @dependabot in dexidp#3724 * build(deps): bump tonistiigi/xx from 1.4.0 to 1.5.0 by @dependabot in dexidp#3705 * build(deps): bump google.golang.org/api from 0.190.0 to 0.196.0 by @dependabot in dexidp#3721 * build(deps): bump docker/build-push-action from 6.5.0 to 6.7.0 by @dependabot in dexidp#3696 * build(deps): bump golang.org/x/oauth2 from 0.21.0 to 0.23.0 by @dependabot in dexidp#3723 * build(deps): bump sigstore/cosign-installer from 3.5.0 to 3.6.0 by @dependabot in dexidp#3685 * build(deps): bump actions/upload-artifact from 4.3.4 to 4.4.0 by @dependabot in dexidp#3720 * build(deps): bump github.com/dexidp/dex/api/v2 from 2.1.0 to 2.2.0 in /examples by @dependabot in dexidp#3734 * build(deps): bump mheap/github-action-required-labels from 5.4.1 to 5.4.2 by @dependabot in dexidp#3735 * build(deps): bump github.com/mattn/go-sqlite3 from 1.14.22 to 1.14.23 by @dependabot in dexidp#3738 * build(deps): bump google.golang.org/grpc from 1.66.0 to 1.66.2 by @dependabot in dexidp#3742 * build(deps): bump distroless/static-debian12 from `42d15c6` to `dcd3f1f` by @dependabot in dexidp#3754 * build(deps): bump anchore/sbom-action from 0.17.0 to 0.17.2 by @dependabot in dexidp#3746 * build(deps): bump github.com/Masterminds/sprig/v3 from 3.2.3 to 3.3.0 by @dependabot in dexidp#3753 * build(deps): bump aquasecurity/trivy-action from 0.24.0 to 0.28.0 by @dependabot in dexidp#3797 * build(deps): bump github/codeql-action from 3.25.15 to 3.26.13 by @dependabot in dexidp#3795 * build(deps): bump golang from 1.23.1-alpine3.20 to 1.23.2-alpine3.20 by @dependabot in dexidp#3775 * build(deps): bump distroless/static-debian12 from `dcd3f1f` to `26f9b99` by @dependabot in dexidp#3766 * build(deps): bump cloud.google.com/go/compute/metadata from 0.5.0 to 0.5.2 by @dependabot in dexidp#3764 * build(deps): bump docker/setup-buildx-action from 3.6.1 to 3.7.1 by @dependabot in dexidp#3781 * build(deps): bump google.golang.org/grpc from 1.66.1 to 1.67.1 in /examples by @dependabot in dexidp#3774 * build(deps): bump docker/build-push-action from 6.7.0 to 6.9.0 by @dependabot in dexidp#3772 * build(deps): bump anchore/sbom-action from 0.17.2 to 0.17.4 by @dependabot in dexidp#3801 * build(deps): bump github.com/prometheus/client_golang from 1.19.1 to 1.20.5 by @dependabot in dexidp#3799 * build(deps): bump golang.org/x/net from 0.29.0 to 0.30.0 by @dependabot in dexidp#3802 * build(deps): bump actions/dependency-review-action from 4.3.4 to 4.3.5 by @dependabot in dexidp#3804 * build(deps): bump anchore/sbom-action from 0.17.4 to 0.17.5 by @dependabot in dexidp#3803 * build(deps): bump sigstore/cosign-installer from 3.6.0 to 3.7.0 by @dependabot in dexidp#3800 * build(deps): bump github.com/mattn/go-sqlite3 from 1.14.23 to 1.14.24 by @dependabot in dexidp#3805 * build(deps): bump github/codeql-action from 3.26.13 to 3.27.0 by @dependabot in dexidp#3806 * build(deps): bump actions/checkout from 4.1.7 to 4.2.1 by @dependabot in dexidp#3785 * build(deps): bump google.golang.org/api from 0.196.0 to 0.203.0 by @dependabot in dexidp#3807 * build(deps): bump actions/checkout from 4.2.1 to 4.2.2 by @dependabot in dexidp#3808 * build(deps): bump actions/dependency-review-action from 4.3.5 to 4.4.0 by @dependabot in dexidp#3814 * build(deps): bump distroless/static-debian12 from `26f9b99` to `3a03fc0` by @dependabot in dexidp#3812 * build(deps): bump actions/setup-go from 5.0.2 to 5.1.0 by @dependabot in dexidp#3809 * build(deps): bump anchore/sbom-action from 0.17.5 to 0.17.6 by @dependabot in dexidp#3817 * build(deps): bump github.com/fsnotify/fsnotify from 1.7.0 to 1.8.0 by @dependabot in dexidp#3822 * build(deps): bump alpine from `beefdbd` to `1e42bbe` by @dependabot in dexidp#3840 * build(deps): bump golang.org/x/oauth2 from 0.23.0 to 0.24.0 in /examples by @dependabot in dexidp#3832 * build(deps): bump golang from 1.23.2-alpine3.20 to 1.23.3-alpine3.20 by @dependabot in dexidp#3834 * build(deps): bump anchore/sbom-action from 0.17.6 to 0.17.8 by @dependabot in dexidp#3858 * build(deps): bump distroless/static-debian12 from `3a03fc0` to `d71f4b2` by @dependabot in dexidp#3839 * build(deps): bump golang from `0974259` to `c694a4d` by @dependabot in dexidp#3863 * build(deps): bump tonistiigi/xx from 1.5.0 to 1.6.1 by @dependabot in dexidp#3879 * build(deps): bump golang.org/x/crypto from 0.28.0 to 0.31.0 in the go_modules group by @dependabot in dexidp#3893 * build(deps): bump golang.org/x/crypto from 0.26.0 to 0.31.0 in /examples in the go_modules group by @dependabot in dexidp#3892 * build(deps): bump github/codeql-action from 3.27.0 to 3.28.0 by @dependabot in dexidp#3898 * build(deps): bump actions/upload-artifact from 4.4.0 to 4.5.0 by @dependabot in dexidp#3890 * build(deps): bump actions/attest-build-provenance from 1.4.3 to 2.1.0 by @dependabot in dexidp#3878 * build(deps): bump golang from 1.23.3-alpine3.20 to 1.23.4-alpine3.20 by @dependabot in dexidp#3866 * build(deps): bump distroless/static-debian12 from `d71f4b2` to `6cd937e` by @dependabot in dexidp#3864 * build(deps): bump actions/dependency-review-action from 4.4.0 to 4.5.0 by @dependabot in dexidp#3862 * build(deps): bump docker/metadata-action from 5.5.1 to 5.6.1 by @dependabot in dexidp#3861 * build(deps): bump aquasecurity/trivy-action from 0.28.0 to 0.29.0 by @dependabot in dexidp#3851 * build(deps): bump gomplate from 4.0.1 to 4.3.0 by @MoeBensu in dexidp#3856 * build: update Go version by @sagikazarmark in dexidp#3913 * build(deps): bump github.com/beevik/etree from 1.4.0 to 1.4.1 by @dependabot in dexidp#3638 * build(deps): bump go.etcd.io/etcd/client/v3 from 3.5.15 to 3.5.17 by @dependabot in dexidp#3842 * build(deps): bump github.com/coreos/go-oidc/v3 from 3.11.0 to 3.12.0 in /examples by @dependabot in dexidp#3903 * build(deps): bump golang.org/x/oauth2 from 0.24.0 to 0.25.0 in /examples by @dependabot in dexidp#3904 * build(deps): bump alpine from 3.20.3 to 3.21.2 by @dependabot in dexidp#3909 * build(deps): bump golang.org/x/net from 0.30.0 to 0.33.0 in the go_modules group by @dependabot in dexidp#3911 * build(deps): bump docker/setup-qemu-action from 3.2.0 to 3.3.0 by @dependabot in dexidp#3914 * build(deps): bump helm/kind-action from 1.10.0 to 1.12.0 by @dependabot in dexidp#3917 * build(deps): bump docker/setup-buildx-action from 3.7.1 to 3.8.0 by @dependabot in dexidp#3916 * build(deps): bump anchore/sbom-action from 0.17.8 to 0.17.9 by @dependabot in dexidp#3915 * build(deps): bump actions/cache from 4.1.2 to 4.2.0 by @dependabot in dexidp#3918 * build(deps): bump golang from 1.23.4-alpine3.20 to 1.23.5-alpine3.20 by @dependabot in dexidp#3928 * build(deps): bump golang.org/x/net from 0.30.0 to 0.34.0 by @dependabot in dexidp#3920 * build(deps): bump distroless/static-debian12 from `6cd937e` to `6ec5aa9` by @dependabot in dexidp#3922 * build(deps): bump golang.org/x/net from 0.28.0 to 0.33.0 in /examples in the go_modules group by @dependabot in dexidp#3912 * build(deps): bump google.golang.org/api from 0.203.0 to 0.217.0 by @dependabot in dexidp#3927 * build(deps): bump google.golang.org/grpc from 1.67.1 to 1.69.4 in /examples by @dependabot in dexidp#3924 * build(deps): bump golang.org/x/net from 0.27.0 to 0.33.0 in /api/v2 in the go_modules group by @dependabot in dexidp#3910 * build(deps): bump google.golang.org/grpc from 1.65.0 to 1.69.4 in /api/v2 by @dependabot in dexidp#3925 * build(deps): bump github.com/stretchr/testify from 1.9.0 to 1.10.0 by @dependabot in dexidp#3931 * build(deps): bump google.golang.org/grpc from 1.69.4 to 1.70.0 in /examples by @dependabot in dexidp#3943 * build(deps): bump actions/upload-artifact from 4.5.0 to 4.6.0 by @dependabot in dexidp#3939 * build(deps): bump oras-project/setup-oras from 1.2.1 to 1.2.2 by @dependabot in dexidp#3936 * build(deps): bump github.com/go-ldap/ldap/v3 from 3.4.8 to 3.4.10 by @dependabot in dexidp#3932 * build(deps): bump docker/build-push-action from 6.9.0 to 6.13.0 by @dependabot in dexidp#3949 * build(deps): bump github/codeql-action from 3.28.0 to 3.28.8 by @dependabot in dexidp#3956 * build(deps): bump github.com/coreos/go-oidc/v3 from 3.11.0 to 3.12.0 by @dependabot in dexidp#3933 * build(deps): bump google.golang.org/protobuf from 1.36.2 to 1.36.4 by @dependabot in dexidp#3947 * build(deps): bump actions/setup-go from 5.1.0 to 5.3.0 by @dependabot in dexidp#3935 * build(deps): bump github.com/beevik/etree from 1.4.1 to 1.5.0 by @dependabot in dexidp#3966 * build(deps): bump anchore/sbom-action from 0.17.9 to 0.18.0 by @dependabot in dexidp#3960 * build(deps): bump mheap/github-action-required-labels from 5.4.2 to 5.5.0 by @dependabot in dexidp#3961 * build(deps): bump actions/attest-build-provenance from 2.1.0 to 2.2.0 by @dependabot in dexidp#3962 * build(deps): bump go.etcd.io/etcd/client/pkg/v3 from 3.5.17 to 3.5.18 by @dependabot in dexidp#3963 * build(deps): bump go.etcd.io/etcd/client/v3 from 3.5.17 to 3.5.18 by @dependabot in dexidp#3965 * build(deps): bump google.golang.org/api from 0.217.0 to 0.219.0 by @dependabot in dexidp#3964 * build(deps): bump golang.org/x/oauth2 from 0.25.0 to 0.26.0 by @dependabot in dexidp#3969 * build(deps): bump golang from 1.23.5-alpine3.20 to 1.23.6-alpine3.20 by @dependabot in dexidp#3972 * build(deps): bump sigstore/cosign-installer from 3.7.0 to 3.8.0 by @dependabot in dexidp#3970 * build(deps): bump golang.org/x/oauth2 from 0.25.0 to 0.26.0 in /examples by @dependabot in dexidp#3968 * build(deps): bump google.golang.org/api from 0.219.0 to 0.220.0 by @dependabot in dexidp#3973 * build(deps): bump google.golang.org/protobuf from 1.36.4 to 1.36.5 by @dependabot in dexidp#3975 * build(deps): bump docker/setup-buildx-action from 3.8.0 to 3.9.0 by @dependabot in dexidp#3977 * build(deps): bump docker/setup-qemu-action from 3.3.0 to 3.4.0 by @dependabot in dexidp#3978 * build(deps): bump golang.org/x/crypto from 0.32.0 to 0.33.0 by @dependabot in dexidp#3980 * build(deps): bump github/codeql-action from 3.28.8 to 3.28.9 by @dependabot in dexidp#3981 * build(deps): bump google.golang.org/api from 0.220.0 to 0.221.0 by @dependabot in dexidp#3985 * build(deps): bump google.golang.org/protobuf from 1.35.1 to 1.36.5 in /api/v2 by @dependabot in dexidp#3976 * build(deps): bump google.golang.org/grpc from 1.69.4 to 1.70.0 in /api/v2 by @dependabot in dexidp#3944 * build(deps): bump github.com/spf13/cobra from 1.8.1 to 1.9.1 in /examples by @dependabot in dexidp#3988 * build(deps): bump golang from 1.23.6-alpine3.20 to 1.24.0-alpine3.20 by @dependabot in dexidp#3984 * Update Go to 1.24 by @sagikazarmark in dexidp#3993 * build(deps): bump alpine from 3.21.2 to 3.21.3 by @dependabot in dexidp#3986 * build(deps): bump golang from `9fed402` to `79f7ffe` by @dependabot in dexidp#3992 * build(deps): bump github.com/spf13/cobra from 1.8.1 to 1.9.1 by @dependabot in dexidp#3987 * build(deps): bump github.com/go-sql-driver/mysql from 1.8.1 to 1.9.0 by @dependabot in dexidp#3991 ## New Contributors * @hur made their first contribution in dexidp#3700 * @koendelaat made their first contribution in dexidp#3598 * @IvoGoman made their first contribution in dexidp#3748 * @dsonck92 made their first contribution in dexidp#3778 * @siarhei-haurylau made their first contribution in dexidp#3793 * @emmanuel-ferdman made their first contribution in dexidp#3755 * @zvlb made their first contribution in dexidp#2941 * @meldsza made their first contribution in dexidp#3676 * @nrwiersma made their first contribution in dexidp#3868 * @JanMkl made their first contribution in dexidp#3930 * @kotx made their first contribution in dexidp#3950 * @bobmaertz made their first contribution in dexidp#3941 **Full Changelog**: dexidp/dex@v2.41.0...v2.42.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
This PR adds resource versions to connectors that are API controlled but default. It does not require the Connector to use the resource revision, so that Kubernetes still functions.
What this PR does / why we need it
This fixes an issue with connectors where updates to a connector via the API does not get applied to the runtime use of the connector. This seems to have been previously fixed for Kubernetes specifically, but this is a more general fix.
Fixes #3855
Special notes for your reviewer
I see no obvious way to test this, but if I am missing something, please let me know.