Comments (20)
@RangelReale would you consider transferring ownership of it @openshift? We (Red Hat) use it extensively in openshift/origin. While we are unlikely to implement any features, we will handle security issues since they would impact our OAuth server.
from osin.
@RangelReale here is my proposed plan, let me know if you agree (also what about RangelReale/osincli?):
- Transfer ownership of this repo to @dobbymoodge
- @dobbymoodge has the ability to transfer repos to @openshift, and will do so once you give him ownership
- Once this repo becomes openshift/osin, @RangelReale will fork it to RangelReale/osin
- @RangelReale will disable issues and PRs on the "new" RangelReale/osin and update the GitHub description to point to openshift/osin
The above should make it so that anyone using RangelReale/osin will not break, they will simply be frozen in time.
All issues and PRs will be left intact and will live at openshift/osin
from osin.
Fine with me, let's do this on monday, if someone has any objection, please say here before that.
from osin.
@RangelReale thanks for your work over the years
from osin.
Amazingly, I was just today studying this project to implement it. But thanks for the heads up! I'll check yours and other's recommendations.
from osin.
Thanks for the work @RangelReale. Sorry that the project is deprecated, but it had its time.
from osin.
@enj definitely, I think this would be the best solution for this. How can we do this?
from osin.
@RangelReale I have reached out to the people who have done similar transfers before. I will update you once I know more.
from osin.
@RangelReale good to go?
from osin.
Sorry for the delay, I was in a place with bad internet, now it is ok. I will do the transfer now, of both osin and osincli.
from osin.
Done, please contact dobbymoodge to accept the transfer.
from osin.
@RangelReale The repository transfers expired. Can you please re-transfer the repo?
I had a busy weekend :(
from osin.
Done.
from osin.
@RangelReale I believe the transfers are complete - I think now you just need to fork them back and freeze.
from osin.
Done. Can I make a notice in my README before freezing?
from osin.
@RangelReale just changing the description would be safer. Otherwise you will have a commit in your fork that does not exist in the openshift/osin code.
from osin.
Going forward, who do we tag to review issues and merge pull requests?
Because I'm about to use this package on a project, and I'll like to know who would be looking at stuff.
I'm happy to contribute, of course, just want to know that someone would be interested in reviewing.
Also, is there any plans to review all the open pull requests and issues?
from osin.
Going forward, who do we tag to review issues and merge pull requests?
The relevant people already watch the repo.
Because I'm about to use this package on a project
Please don't. There are far better and more robust options out there such as ory/fosite, dex, etc.
I'm happy to contribute, of course, just want to know that someone would be interested in reviewing.
That effort is better spent on a more active project.
Also, is there any plans to review all the open pull requests and issues?
Unless it is a security issue, I will likely close them.
As I noted above, we are not likely to implement any features. The repo is effectively in maintenance mode. The core of this repo is on the order of 650 LOC and likely does not suffice for most use cases. Also, please use OIDC as that adds a lot of the missing pieces to OAuth 2. I cannot think of any good reason for someone starting a new project to use this repo.
from osin.
Okay. Thank you.
from osin.
Well, the process is done, so I am closing the issue.
from osin.
Related Issues (20)
- CheckBasicAuth does not url decode the client_id and client_secret HOT 2
- How should I connect with my own user table ? HOT 2
- Expires time not correct HOT 4
- password grant type HOT 2
- Support for UMA, a.k.a. resource sets? HOT 5
- The redirect URI is checked to be not empty for every flow HOT 4
- grpc HOT 1
- Bot sending repeated messages HOT 5
- Can we fork this repo like gogs and gitea? HOT 3
- refactor example imports HOT 6
- refresh token expiration HOT 4
- in some case should check err == osin.ErrNotFound HOT 4
- Future Release Branches Frozen For Merging | branch:release-4.17 branch:release-4.18 HOT 2
- the project active? HOT 6
- Depends on "github.com/RangelReale/osin" HOT 2
- Authorization Code Request Handler always expects client secret to be present HOT 8
- RedirectURI prefix validation HOT 2
- grant type CLIENT_CREDENTIALS HOT 10
- The vulnerability CVE-2021-4294 has been fixed, but no specific tag denotes the patched version. HOT 5
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 osin.