oauth2_proxy
oauth2_proxy copied to clipboard
bitly
Support for group restriction for Gitlab provider
@jehiah any plans merging this? I would love to see this merged and released.
I tested this feature and it works great, except if a user is a member of a lot of groups (or if they're an admin, because in that case the GitLab API returns all the groups, not just those the user is a member of). The GitLab API paginates the list of groups, so I had to add c141ff8da3ce7d865db38ccd4e5c157727a6b0d5 to make it work with more than 20 groups.
So this pull request will be included in the next release? When will that be approximately?
We would also appreciate this feature, without this we cannot use gitlab authentication. I vote for merge of this feature.
Been using this in production and I can say that the code has been stable for my setup.
I've also been running this in production (with c141ff8 applied) for a while now, and it's working as expected.
@JacobAae, do you want to add c141ff8 to this PR, if it looks good to you? Otherwise, I can open a new PR, but I think it makes more sense to keep everything in one place.
Let me know if there's anything I can do to help get this merged.
@BenoitKnecht I've created a patch from your commit and applied to include it in the merge request, as it looks fine. I however doubt this will ever be merged, it has been pending for years, but I know several places where it runs in production.
Fyi: there is an active discussion about forking this (obviously unmaintained) project here: #628
