claudyus / LXC-Web-Panel

LXC Web Panel improved for lxc 1.0+
http://claudyus.github.io/LXC-Web-Panel/
Other
221 stars 68 forks source link

Add htpasswd authentication backend #17

Closed ghost closed 10 years ago

ghost commented 10 years ago

I haven't tested it properly, so this is just a quick hack to see how much problem would it be to add htpasswd file backed up authentication and if it would be possible to merge it upstream.

Reasoning: if I understand correctly, currently you have to put on every LXC node separate instance of the LWP. Editing and distributing htpasswd file for multiple machines is a little bit easier (if you are using something like puppet) then using sqlite db files.

claudyus commented 10 years ago

Hi mihu, your pull request is welcome!

I like the idea to use the htpasswd as auth backend to provide an easy way to manage user account with puppet!

I will test and merge it soon, meanwhile can you add some words about this auth type inside readme.

Bye Claudio

ghost commented 10 years ago

I've changed the README as well.

ghost commented 10 years ago

Fixed the typos. Btw, how do you run it. I've try to directly launch the bin/lwp, but it can't find lxclite (and after adjusting PYTHONPATH it cannot find lwp:). Building deb fails on pyversions: error parsing Python-Version attribute. Any hints?:)

claudyus commented 10 years ago

Hi @mihu,

try to rebase your commit over master branch, not app-rework

Than you can use sudo ./lwp.py --debug

can you also change if (condition): to if condition: in your code?

ghost commented 10 years ago

I'me managed to build the package by adjusting the X-Python-version. I've hit a several other issues, so more patches are comming:)

claudyus commented 10 years ago

Thanks for your contribute, anyway base your future work on the master branch don't use the app-rework as base code.

ghost commented 10 years ago

Sorry, but I will be deploying more polished app-rework, hence no use for basing patches against master.

claudyus commented 10 years ago

Included in master, I will merge it in app-rework later

claudyus commented 10 years ago

I just merge app-rework in master where this code is included. Thanks