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.

Sign up for GitHub

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

Jump to bottom

connections: remove connections API #265

Closed
aeneasr opened this issue Sep 23, 2016 · 1 comment
Closed

connections: remove connections API #265

aeneasr opened this issue Sep 23, 2016 · 1 comment
Labels
breaking change Changes behavior in a breaking manner. feat New feature or request.
Milestone
0.6.0

Comments

@aeneasr
Copy link
Member

aeneasr commented Sep 23, 2016 •
edited
Loading

I think the social connections API is adding confusion around what hydra is doing and what not. Additionally, I don't think anyone really uses it.

vote 👍 to remove or 👎 to keep
@aeneasr aeneasr added this to the unplanned milestone Sep 23, 2016
@janekolszak
Copy link

This API is a mapping between local userid and remote userid.

This mapping would be used in the IdP, keeping this in hydra is just one request more for IdP.

@aeneasr aeneasr modified the milestones: 0.6.0, unplanned Oct 9, 2016
@aeneasr aeneasr added feat New feature or request. breaking change Changes behavior in a breaking manner. and removed discuss labels Oct 9, 2016
aeneasr pushed a commit that referenced this issue Oct 17, 2016
* oauth2/introspect: make endpoint rfc7662 compatible - closes #289
cde615d 
* connections: remove connections API - closes #265
* oauth2: token revocation endpoint - closes #233
* vendor: update to fosite 0.5.0
aeneasr pushed a commit that referenced this issue Oct 17, 2016
connections: remove connections API - closes #265
ffd5907
aeneasr pushed a commit that referenced this issue Oct 17, 2016
connections: remove connections API - closes #265
bc7f2b6
aeneasr pushed a commit that referenced this issue Oct 17, 2016
connections: remove connections API - closes #265
a56f15d
@aeneasr aeneasr closed this as completed Oct 24, 2016
aeneasr pushed a commit that referenced this issue Oct 25, 2016
@arekkas
0.6.0 (#293)
8256356 
* oauth2: scopes should be separated by %20 and not +, to ensure javascript compatibility - closes #277
* oauth2/introspect: make endpoint rfc7662 compatible - closes #289
* warden: make it clear that ladon.Request.Subject is not required or break bc and remove it - closes #270
* travis: execute gox build only when new commit is a new tag - closes #285
* docs: improve introduction (#267)
* core: (health) monitoring endpoint - closes #216
* oauth2/introspect: make endpoint rfc7662 compatible - closes #289
* connections: remove connections API - closes #265
* oauth2: token revocation endpoint - closes #233
* vendor: update to fosite 0.5.0
* core: add sql support #292
* connections: remove connections API - closes #265
* all: coverage report is missing covered lines of nested packages - closes #296
* cmd: prettify the `hydra token user` output - closes #281
* travis: make it possible for travis-ci to build forked repos - closes #295
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking change Changes behavior in a breaking manner. feat New feature or request.
Projects
None yet
Milestone
0.6.0
Development

No branches or pull requests
2 participants
@janekolszak @aeneasr