Comments (4)
Sounds like a good idea - i would not oppose a PR which would create like a bootstrap
sub modules which holds the logic currently found in the cmd itself (minus the commandline parsing and environment variable logic).
Since lot of things need to happen to startup the services provided by konnectd
it is not particularly trivial to collect it all and it seems bad if embedding could would need to replicate it all.
from konnect.
As #19 is nearing completion we would like to propose some follow up changes to further improve embedding.
Configuration files/keys
Instead of storing the path to the file the bootstrap should contain the unmarshaled config struct. With this change the consumer can decide how the load/un-marshall the config during the boot-process (etcd, hard-coded, different format). This would also decouple the business-logic from any io which should also improve test-ability.
This will require some concept on how to add extension-points to the bootstrap. (Extend interface introduced in #19 ?)
Assets (index.html)
Similar as above but here the path to index.html would be replaced by an io.reader of to allow loading assets from an vfs or do some other processing. Identifier-webapp should be optional as it might be hosted on another server.
from konnect.
As #19 is nearing completion we would like to propose some follow up changes to further improve embedding.
Would you please make a ticket for this - so it can be properly discussed there instead of here. Thanks!
from konnect.
This was implemented, since #19 was merged.
from konnect.
Related Issues (15)
- Access tokens use algo 'PS256' which seems not to be supported/implemented by any token verifier .... HOT 7
- Authentication via SAML IDP possible? HOT 18
- `--log-level` not available for `cookie` ? HOT 2
- Is there any way to map the user information user info as returned via the userinfo_endpoint to a user in ldap? HOT 4
- Further improve usage as library HOT 1
- Make identifier-webapp optional
- [Spec] Graph Backend HOT 1
- Add eye icon for password field
- subject is based on the ldap dn, which is neither stable, nor non-reassignable HOT 3
- SAML Allow for issuer != MetadataEndpoint HOT 3
- redirect_uri's require trailing slash on logout? HOT 2
- Commit f8c1f4a (current top) does not build HOT 10
- Get memberOf LDAP property as Claim attribute of JWT token. HOT 1
- SIGSEGV on startup due to uri parse error in identifer-registration.yaml HOT 1
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 konnect.