Skip to content

CloundFoundry connector#1624

Open
xtremerui wants to merge 23 commits intodexidp:masterfrom
concourse:pr/add-cf-connector-sync
Open

CloundFoundry connector#1624
xtremerui wants to merge 23 commits intodexidp:masterfrom
concourse:pr/add-cf-connector-sync

Conversation

@xtremerui
Copy link
Copy Markdown

Support for authenticating against CloudFoundry

Documentation of CloudFoundry UAA server

cc @pivotal-jwinters

@julia-pu julia-pu force-pushed the pr/add-cf-connector-sync branch 3 times, most recently from 47c7199 to 556cf89 Compare January 13, 2020 18:56
@xtremerui
Copy link
Copy Markdown
Author

Hi, just checking back. Since there is no feedback here yet so I am wondering are there certain requirements to meet for Dex to add a new connector? Is there a way to move forward for this PR or its just a simple Nah not this connector?

Thank you for your time.

@Albertoimpl
Copy link
Copy Markdown

Our team is also interested in this connector to adopt dex.
I saw it used to be supported but seems to be gone now: https://github.com/dexidp/dex/pull/542/files
Is there anything we can do to move this forward?
Thanks!

@mvdkleijn
Copy link
Copy Markdown
Contributor

@xtremerui This also has merge conflicts... 😄

@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch 2 times, most recently from efa3156 to ed22ce5 Compare August 4, 2020 19:02
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch 2 times, most recently from 526f2ed to 5549159 Compare September 28, 2020 19:39
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch 3 times, most recently from c7bbcf4 to 4e238e8 Compare October 5, 2020 17:39
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from 4e238e8 to 3f8cdd5 Compare October 16, 2020 16:43
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch 3 times, most recently from 8d44c14 to d2b321f Compare November 5, 2020 04:17
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from d2b321f to 7d9e841 Compare December 15, 2020 15:54
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from 7d9e841 to f3d4b96 Compare March 5, 2021 17:18
@xtremerui
Copy link
Copy Markdown
Author

xtremerui commented Mar 5, 2021

@sagikazarmark I kown there must be lots of pending PRs in your list. Just wanted to get some feedback here so we could move forward. Thx!

We assume that a PR to dexidp/website to include the doc is needed. Then we will update README.md here.

@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from a4efbec to 73af7c0 Compare March 12, 2021 22:28
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from 73af7c0 to 75f4a22 Compare March 20, 2021 20:05
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from 75f4a22 to fabca39 Compare June 29, 2021 15:07
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from fabca39 to 5b5cd80 Compare August 3, 2021 13:18
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch 2 times, most recently from 632cdf8 to acc2c20 Compare November 16, 2021 00:07
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from acc2c20 to ee1e41e Compare December 1, 2021 14:55
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch 2 times, most recently from 7d0ae05 to aa82d4f Compare February 8, 2022 20:43
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from 6f2b2a0 to 014564c Compare September 1, 2025 13:24
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from 014564c to 59511a0 Compare February 23, 2026 15:13
@nabokihms
Copy link
Copy Markdown
Member

I assume it fails because we changed the interface.

@xtremerui
Copy link
Copy Markdown
Author

@taylorsilva I assume we don't need Concourse to support Cloudfoundry anymore? Do we still looking to add this connector to Dex?

@taylorsilva
Copy link
Copy Markdown
Contributor

We do still need this. Last big update was a year ago here: concourse#148

I get that Dex may not want this though.

@nabokihms
Copy link
Copy Markdown
Member

@taylorsilva, this is not the means of wanting something. We want all the connectors! We just don't have the capacity to support them all. I personally have no CloudFoundry in my space, and I do not know any real clients (which doesn't mean there are not any, I just don't know them). Which means neither I nor any other maintainers can effectively answer community questions regarding CloudFoundry.

My desire is that instead of accepting all the connectors, give users the way to easily add their own connectors.

The initial issue: #1907
The latest proposal: #4578

matzegebbe and others added 23 commits March 3, 2026 14:19
… interface (dexidp#4589)

The PKCE support added in v2.45.0 changed the CallbackConnector interface
signatures but missed updating the authproxy and oauth connectors. This
caused a type assertion failure in handleConnectorLogin(), resulting in
"Requested resource does not exist" errors when using these connectors.

Update LoginURL to return (string, []byte, error) and HandleCallback to
accept a []byte connData parameter for both connectors and their tests.

Signed-off-by: Mathias Gebbe <mathias.gebbe@gmail.com>
`groups` is a reserved word in MySQL >= 8.0.2, causing migration 13
to fail with a syntax error on `ALTER TABLE password ADD COLUMN groups`.

Fixes dexidp#4579

Signed-off-by: Michiel De Backker <mail@backkem.me>
- Verifies user is part of orgs and spaces for group claims

Signed-off-by: Joshua Winters <jwinters@pivotal.io>
Co-authored-by: Shash Reddy <sreddy@pivotal.io>
Co-authored-by: Topher Bullock <tbullock@pivotal.io>
Signed-off-by: Josh Winters <jwinters@pivotal.io>
Signed-off-by: Daniel Lavoie <dlavoie@live.ca>
Signed-off-by: Joshua Winters <jwinters@pivotal.io>
Co-authored-by: Rui Yang <ryang@pivotal.io>
Co-authored-by: Rui Yang <ryang@pivotal.io>
Signed-off-by: Joshua Winters <jwinters@pivotal.io>
Signed-off-by: Zoe Tian <ztian@pivotal.io>
Co-authored-by: Ciro S. Costa <cscosta@pivotal.io>
Signed-off-by: w3tian <w3tian@uwaterloo.ca>
Signed-off-by: Rui Yang <ryang@pivotal.io>
Co-authored-by: Joshua Winters <jwinters@pivotal.io>
Signed-off-by: Joshua Winters <jwinters@pivotal.io>
Co-authored-by: Rui Yang <ryang@pivotal.io>
gofumpt-ed

Signed-off-by: Rui Yang <ryang@pivotal.io>
Signed-off-by: Rui Yang <ruiya@vmware.com>
Signed-off-by: Rui Yang <ruiya@vmware.com>
Signed-off-by: Rui Yang <ruiya@vmware.com>
Signed-off-by: Rui Yang <ruiya@vmware.com>
Signed-off-by: Kump3r <tonevkalin@gmail.com>
Signed-off-by: Kump3r <tonevkalin@gmail.com>
…ructure

Signed-off-by: Kump3r <tonevkalin@gmail.com>
Signed-off-by: Kump3r <tonevkalin@gmail.com>
Signed-off-by: IvanChalukov <ichalukov@gmail.com>
Signed-off-by: Kump3r <tonevkalin@gmail.com>
Signed-off-by: Taylor Silva <dev@taydev.net>
@xtremerui xtremerui force-pushed the pr/add-cf-connector-sync branch from 59511a0 to cc648ec Compare March 3, 2026 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connector release-note/new-feature Release note: Exciting New Features

Projects

None yet

Development

Successfully merging this pull request may close these issues.