Skip to content

makes middlewares pluggable. i.e for custom oauth flows or analytics etc.#395

Open
fredr wants to merge 1 commit into
rlidwka:masterfrom
soundtrackyourbrand:add-middleware-plugin
Open

makes middlewares pluggable. i.e for custom oauth flows or analytics etc.#395
fredr wants to merge 1 commit into
rlidwka:masterfrom
soundtrackyourbrand:add-middleware-plugin

Conversation

@fredr

@fredr fredr commented Apr 14, 2016

Copy link
Copy Markdown

I'm building a github web oauth authorization plugin for sinopia. To achieve that in a good way I need to be able to register custom routes in sinopia, so that I don't have to have github or sinopia secrets on my client to be able to generate a .npmrc

@rbecheras

Copy link
Copy Markdown

👍 💯 ;-)

A github web oauth authorization plugin for sinopia ?

Awesome !

Even better, what about a provider-agnostic web oauth authorization plugin for sinopia ?

And then, on top of this, you could build a github adapter for this plugin, and any else would be able to build adapters for other providers (like gitlab for example), or private ones (for private Oauth2 servers).

Personally, if it would exist, I will use all of these in my organisation. I think I'm not the only one person that could benefit of a such capabilities.

@rbecheras

Copy link
Copy Markdown

@fredr , could you please post a link to the github repo of your plugin project (if exists) ?

@dgaya

dgaya commented Apr 15, 2016

Copy link
Copy Markdown

Neat !!
I'm afraid this repo is no longer supported.

Do you mind PR to our clone https://github.com/fl4re/sinopia ?
I already have a google and github oauth2 login in our branch OAuth_login.
But your solution is neat, I will use it.

@fredr

fredr commented Apr 15, 2016

Copy link
Copy Markdown
Author

@rbecheras sure, this would enable anyone to write such plugins. Im building this one: https://github.com/soundtrackyourbrand/sinopia-github-oauth
Its a bit work in progress, but first version should be done today, or soon enough.

@dgaya I know, but I thought I'd add a PR anyway. Cool, your fork seems to be more active! I'll open a PR there!

@JDvorak

JDvorak commented Apr 27, 2016

Copy link
Copy Markdown

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants