Comments (31)
I suggest the following:
- move this repo here to the omniauth group as it is somehow the best maintained and visible fork(I contributed to as well), I can add you @m0n9oose as owner there
- adding @m0n9oose @hhorikawa @stanhu there as maintainers
- @m0n9oose provides push rights for @hhorikawa and @stanhu for the gem
- all are gonna use the PR workflow, no self-merge
- bring in the changes from the gitlab folks in soon to release an updated version containing those changes via the extisting gem name
- joint development in the way open source should work
WDYT?
from omniauth_openid_connect.
Welcome to the omniauth family! ❤️ I maintain the omniauth-identity gem. Please ping me if I can be of help!
from omniauth_openid_connect.
@bufferoverflow transfer is completed. omniauth-oidc team has been added as maintainer
from omniauth_openid_connect.
just ping me as soon as you came to a common agreement, I created the omniauth org and can create a project there and add maintainers.
@stanhu mabye also interesting for you
from omniauth_openid_connect.
@stanhu @hhorikawa i need your email addresses to add you as owners to rubygems
Thank you!
I'll provide my restructured separated PRs to the repo at the new space.
My email is hidden.
https://rubygems.org/profiles/hhorikawa
from omniauth_openid_connect.
@stanhu I think this makes sense as @alexkalderimis and @dblessing are working on the omniauth topics on your end. Maybe worth if they could start with first with some MRs here, so the other maintainers could build up trust. Maybe the gitlab-ci file you already had? I've setup the ci already here: https://gitlab.com/omniauth/omniauth_openid_connect
btw. why this is important for me: https://gitlab.com/gitlab-org/gitlab-foss/-/merge_requests/8018 and follow-up https://gitlab.com/gitlab-com/gl-infra/infrastructure/-/issues/2804
from omniauth_openid_connect.
Good evening,
I would like to propose that I will take over the project if m0n9oose-san don't mind.
I maintain this repository:
https://github.com/netsphere-labs/omniauth-openid-connect/
This repository has already merged with the head of @m0n9oose/omniauth_openid_connect.
So then, please permit me to upload gem packages to
https://rubygems.org/gems/omniauth_openid_connect/
Thank you!
from omniauth_openid_connect.
By the way, how about talking to people from @omniauth and moving the project under their namespace while simultaneously moving to group maintenance?
from omniauth_openid_connect.
@bufferoverflow Thanks for the ping!
We've made significant improvements to this gem in https://gitlab.com/gitlab-org/gitlab-omniauth-openid-connect. I'd suggest starting with that.
See https://gitlab.com/gitlab-org/gitlab-omniauth-openid-connect/-/blob/master/CHANGELOG.md for more details.
from omniauth_openid_connect.
@hhorikawa are you asking me to allow you to push your code from a completely different repository to rubygems.org using the same gem name as this repository? Sorry, but this sounds way too dangerous, considering there are a lot of projects that use this gem.
If you have any proposals like improvements/fixes - you're welcome to open a PR.
from omniauth_openid_connect.
@stanhu @hhorikawa i need your email addresses to add you as owners to rubygems
from omniauth_openid_connect.
@m0n9oose hidden. Thanks!
from omniauth_openid_connect.
Done to update my README.
from omniauth_openid_connect.
@hhorikawa please enable 2FA on your rubygems account
from omniauth_openid_connect.
@davidwessman be patient. Your efforts are appreciated, and PRs will be reviewed as soon as possible. Please keep in mind that we work on this in our free time when we have it.
from omniauth_openid_connect.
Thanks all! I would welcome help upstreaming my changes in https://gitlab.com/gitlab-org/gitlab-omniauth-openid-connect.
In the interest of transparency, I've added a few colleagues with the Triage
role to help with reviews:
We should probably have some lightweight process for giving merge access to those who actively help with this project. I welcome ideas here.
from omniauth_openid_connect.
@Eric-Guo Please include anyone else who might be concerned.
from omniauth_openid_connect.
At the very least it would be great if the current Git snapshot could be published, as the current last version is missing OmniAuth 2 support.
from omniauth_openid_connect.
What about the gem name?
Which people could/should have access to push the gem?
@m0n9oose What do you think?
from omniauth_openid_connect.
Cc @jjbohn, who I reckon still has upload rights for the original gem.
from omniauth_openid_connect.
hey guys. There are a lot of different proposals, which makes me a bit confused about what actually is going on.
from omniauth_openid_connect.
@m0n9oose, I guess I am to blame since I turned this discussion that way 😄
I guess the ultimate aim is to have One Good OpenID Connect strategy for OmniAuth.
Probably the best way to achieve that would be to go through all forks and merge them together, and then probably move the project under @omniauth and maybe group-maintain it: the maintainers of existing forks (@m0n9oose, @hhorikawa, @stanhu) could probably work as a team.
What do you think?
from omniauth_openid_connect.
@andrewshadura sounds great to me. Do we have anybody from @omniauth here? Is it possible to move one of the forks under your namespace?
from omniauth_openid_connect.
I understand @bufferoverflow is.
from omniauth_openid_connect.
@andrewshadura ah, yeah, sorry I missed that. Great.
@bufferoverflow can we just move existing repository under your namespace?
from omniauth_openid_connect.
I made @m0n9oose an owner to start the transfer, also added https://github.com/orgs/omniauth/teams/omniauth-oidc with the maintainer crew ( @m0n9oose @hhorikawa @stanhu ).
from omniauth_openid_connect.
btw. I also made https://gitlab.com/omniauth long time ago so we can setup gitlab-ci if needed. omniauth/omniauth-org#3
from omniauth_openid_connect.
Next idea: talk @jjbohn into uploading an empty gem omniauth-openid-connect
depending on this one 🙂 Or the other way around.
from omniauth_openid_connect.
I am so glad that my issue got this much traction! 🙂
from omniauth_openid_connect.
I have opened a few PRs during the past week, it feels bad to @mention to get any attention to it - or is that the way to go?
from omniauth_openid_connect.
FYI, I filed a request with GitHub Support to disconnect this project form the original fork. That way pull requests will go directly to this project instead of the original project. I found it a bit confusing that new pull requests would default to the other project. 😄
from omniauth_openid_connect.
Related Issues (20)
- Allow relaxing state check for IdP initiated SSO HOT 6
- Automatically set (and send?) redirect_uri HOT 2
- Dynamically Set ACR Values HOT 3
- OneLogin OIDC post_logout_redirect_uri issue HOT 3
- OpenID-provider without `userinfo_endpoint`
- Uninitialized constant json::jws::unknownalgorithm HOT 1
- Pitfalls setting up OIDC with ADFS HOT 2
- Why should the logout path be relative to request_path
- Possible bug when upgrading to 0.7.0 and openid_connect to 2.2.0 HOT 2
- When using jwks_uri, default value fails becuase it's not a URI
- OmniAuth::Strategies::OpenIDConnect::CallbackError, csrf_detected | Invalid 'state' parameter HOT 3
- Authentication failure! no implicit conversion of Hash into String (version 0.6.1) HOT 3
- Migration guide from gitlab-omniauth-openid-connect to this gem? HOT 4
- Problem using microsoft oauth2 as provider because of dynamic issuer HOT 22
- Dynamic client_options.redirect_uri value HOT 3
- Could not authenticate you from [My Provider name] because "Unknown" HOT 3
- Actioncontroller::InvalidAuthenticityToken with omniauth_openid_connect and omniauth-rails_csrf_protection HOT 1
- How not to send the `state` parameter? HOT 21
- Getting a routing error after initialization HOT 3
- Back-channel Single Sign Out Support
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from omniauth_openid_connect.