Agrando / IdentityServer.Contrib.RavenDB

RavenDB stores for Identityserver v3
0 stars 0 forks source link

IdentityServer4 support #4

Closed lahma closed 3 years ago

lahma commented 3 years ago

I found your project while trying to find a RavenDB compatible store for IdentityServer4. I forked the repo and tried to do quick port but it seems that the task is a bit more complicated than that with APIs and stores being different. It might be even hard to maintain backwards compatibility with JSON schema.

Should this project create a new repo IdentityServer4.Contrib.RavenDB and create a new revisited implementation or are you more on maintenance mode with this? I could also create such new repo and publish on NuGet, but I don't want to step on anyone's toes. This repository also doesn't define a license file so that is currently a blocker for that idea (would be good to have one anyway).

What are your thoughts?

bjarteskogoy commented 3 years ago

I no longer work at the company. But I'm still a member of the organisation on Github. The lack of a license can be seen as the most permissive license you can find.

A new repo for IdentityServer4 would be the right thing. And maybe also support for RavenDB 4. There is a branch in the repo where I started out on RavenDB 4 support.

Feel free to create a new repo and publish it to NuGet.

bjarteskogoy commented 3 years ago

Getting a proper build pipeline with testing up and running would be a massive improvement too.

lahma commented 3 years ago

Thank you for getting back so promptly. I'll see what I can do as part of our Identity IdentityServer4 migration 👍🏻

lahma commented 3 years ago

And just found about https://github.com/ravendb/IdentityServer4.Contrib.RavenDB , so this might be the easiest path as it's coming straight from the source. Thanks again for guidance and information!