New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ENC support? #134
Comments
We don't state that it's a Puppet Dashboard replacement:
It's only aiming to replace the reporting part and it's solely based on PuppetDB. Since PuppetDB doesn't offer ENC capabilities, neither do we. |
understood, thanks! |
Would you be opposed to me and my colleagues adding ENC functionality to PuppetBoard? Via a pull request at some point? |
I think having a community web-enc is a good thing. I'm not sure I like the idea of bolting it on to puppetboard. We're behind on prs and features as it is. One concern I have is auth. Puppetboard is simple because it doesn't have an auth system.... if you can hit puppetboard then you can view puppet reports. Adding auth would seem to be a requirement for an ENC. There is a plugin effort in place on another PR(#168), that possibly could be leveraged to build out ENC functionality. Ideas:
|
I think 3 is the best option in the short term. I've commented on the issue regarding plugins as well - suggesting Flask Blueprints. Adding authentication to puppetboard should be very easy if we rely on LDAP, of course not everybody will have an LDAP directory to use. I'd be happy to work on adding auth if nobody else has done so? I think auth should be in the core in my opinion, starting with LDAP support. I've already got working LDAP/Flask authentication in several of UoS's (my employers) Flask apps. |
I don't want to see custom auth/login/session code or anything hardcoded to any authentication backend. That's a recipe for disaster security wise and a problem later on extension wise. Things like Flask-Login can handle a myriad of authentication providers, from LDAP through Basic/Digest and oauth and integrate well with other packages providing authorisation like Flask-Principal. |
Small spelling changes
since this is described as a "puppet-dashboard replacement", are there plans to also implement the ENC capabilities of puppet-dashboard here?
The text was updated successfully, but these errors were encountered: