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

CloundFoundry connector #1624

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

xtremerui
Copy link

Support for authenticating against CloudFoundry

Documentation of CloudFoundry UAA server

cc @pivotal-jwinters

@xtremerui
Copy link
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

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
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 3 times, most recently from 8d44c14 to d2b321f Compare November 5, 2020 04:17
@xtremerui
Copy link
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 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 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 2 times, most recently from b08eada to 177adac Compare October 5, 2022 03:22
@xtremerui
Copy link
Author

@sagikazarmark the PR is updated. Thx for the reviewing.

We will submit a PR to dex docs too. BTW, in the readme https://dexidp.io/docs/connectors/oauth/ is somehow broken as in the connector listing, though the PR for oauth doc is already merged.

nabokihms and others added 15 commits August 7, 2024 19:45
* Use scheme without :// suffix
* Make test ldap server listen on custom ports to avoid stepping into go-ldap defaults

Signed-off-by: m.nabokikh <[email protected]>
Workaround to run gomplate from a non-root directory in distroless images, because gomplate tries to access CWD on start.
See: hairyhenderson/gomplate#2202

Signed-off-by: m.nabokikh <[email protected]>
- Verifies user is part of orgs and spaces for group claims

Signed-off-by: Joshua Winters <[email protected]>
Co-authored-by: Shash Reddy <[email protected]>
Co-authored-by: Topher Bullock <[email protected]>
Signed-off-by: Josh Winters <[email protected]>
Signed-off-by: Joshua Winters <[email protected]>
Co-authored-by: Rui Yang <[email protected]>
Co-authored-by: Rui Yang <[email protected]>
Signed-off-by: Joshua Winters <[email protected]>
Signed-off-by: Zoe Tian <[email protected]>
Co-authored-by: Ciro S. Costa <[email protected]>
Signed-off-by: w3tian <[email protected]>
Signed-off-by: Rui Yang <[email protected]>
Co-authored-by: Joshua Winters <[email protected]>
Signed-off-by: Joshua Winters <[email protected]>
Co-authored-by: Rui Yang <[email protected]>
gofumpt-ed

Signed-off-by: Rui Yang <[email protected]>
Signed-off-by: Rui Yang <[email protected]>
Signed-off-by: Rui Yang <[email protected]>
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.

6 participants