clonos / control-pane

ClonOS WEB control panel (CBSD WEB UI)
https://clonos.tekroutine.com
133 stars 33 forks source link

* Merged shell-escape-fix #32

Closed bozhinov closed 3 years ago

bozhinov commented 3 years ago
bozhinov commented 3 years ago

@olevole I would check lines 1387 & 2003 from clonos.php during the refactoring I believe I ve uncovered some bugs just does not seem right

olevole commented 3 years ago

@bozhinov lines 2003: do you mean this line? https://github.com/clonos/control-pane/blob/aef78b712561759469ab09d713cff967b6db212d/php/clonos.php#L2003

This is an intermediate/temporary SQLite3 file that is used for dynamic parameters for puppet module. When a container is created with filled parameters of a third-party service, the puppet module works inside the container and makes all the settings. This SQlite3 file is needed to bootstrap such a container

bozhinov commented 3 years ago

No, I ve created separate issues both I m also in the process of installing a test bed so I can test the entire refactoring I ve done.

olevole commented 3 years ago

checking the proposed code in a working environment is very much appreciated))

bozhinov commented 3 years ago

if I can get it to work. The info in the distfile for the cbsd puppet plugin is incorrect for starters

bozhinov commented 3 years ago

Please help me get ClonOS running, so I can test this. All problems I encounters have been recorded here

olevole commented 3 years ago

@bozhinov sorry for the delay. Can you write down the details, what exactly is the problem? I think in the next week I will be able to update the image and instructions.

bozhinov commented 3 years ago

I tried installing ClonOS in a VM so I can test my commits I followed the instructions and I failed All issues I encountered I recorded as issues here and in ports and www repos

bozhinov commented 3 years ago

@olevole This has been tested and is ready for use

olevole commented 3 years ago

Now everything seems to be in order in the code. However, for some reason the system writes the conflict to the php/clonos.php file and the merge is blocked. I am trying to understand why

bozhinov commented 3 years ago

What's failing ? I have a test setup now so I can fix it in no time

bozhinov commented 3 years ago

I would suggest you copy all files from my repo->fix branch and copy them over your master branch then commit them yourself. that will do it

olevole commented 3 years ago

@bozhinov done, thanks for your work and improvements. We plan to release a new ClonOS build after FreeBSD 13.0-RELEASE ( https://www.freebsd.org/releases/13.0R/schedule/ )

bozhinov commented 3 years ago

Perfect. I plan on helping a bit more with this as I m looking to replace ESXi 6.5 (EOL Dec 2021)

olevole commented 3 years ago

We only have 9 months left to write another ESXi, we need to strain ;)

bozhinov commented 3 years ago

It's more like 2 months. Until the weather gets warmer and then it is off to the garden for me. Those tomatoes ain't gonna grow themselves :)