crkn-rcdr / upholstery

Authorization and abstraction layer for c7a CouchDB databases
0 stars 0 forks source link

Replacing current usage of Upholstery for D10n. #8

Closed RussellMcOrmond closed 4 years ago

RussellMcOrmond commented 4 years ago

https://github.com/crkn-rcdr/d10n is currently offered from via Apache wrapped with upholstery in order to check the JWT.

As upholstery can become much more useful as an API provider for clients #2 , we will want to set up something different for d10n.

RussellMcOrmond commented 4 years ago

We can choose one of the ideas discussed, or think of something different:

SaschaAdler commented 4 years ago

Your first option seems the easiest to set up, and what I'm inclined to do quickly.

RussellMcOrmond commented 4 years ago

To keep the development environments as similar to production as possible, I suggest rather than a proxy that we use something such as:

This way upholstery itself can be used during maintenance of these scripts, rather than a separate Apache configuration.

Unless there is opposition to this option, I'll volunteer to take this on.

RussellMcOrmond commented 4 years ago

https://github.com/crkn-rcdr/upholstery/pull/11 and https://github.com/crkn-rcdr/d10n/pull/10 have the changes required to serve the d10n demo files as static files from Upholstery.

It is currently configured to use https://github.com/crkn-rcdr/d10n as a subproject. That repository is private, and because it historically contained passwords we may want to keep it that way. This repository is public. That mixture may not be desirable:

RussellMcOrmond commented 4 years ago

When discussing having a private subproject, it was decided to not worry about it as it was unlikely anyone else was cloning this project. If this changes, we can adjust accordingly.