framework icon indicating copy to clipboard operation
framework copied to clipboard

fix: docker extension removal

Open Eveeifyeve opened this issue 8 months ago • 3 comments

I added a flag because exporting in the docker container is not available.

$ docker compose exec panel export USER_CONFIRMED_REMOVAL="yes"
OCI runtime exec failed: exec failed: unable to start container process: exec: "export": executable file not found in $PATH: unknown

it is not found in /bin there should be a path /bin/export that should be there to export env, but there is not that's why it fails.

via using this docker image: https://github.com/BlueprintFramework/docker/blob/Master/docker-compose.yml

I also mentioned this in the discord I was told by @prplwtf there will be a flag implemented soon, but I need this asap so here is the pr. Link to message

Eveeifyeve avatar Mar 15 '25 08:03 Eveeifyeve

Flag aside, you can set environmental variables in a Docker container exec command: https://docs.docker.com/reference/cli/docker/container/exec/#env

Loki-101 avatar Mar 15 '25 17:03 Loki-101

I highly doubt this will parse correctly. The -user-confirmed-removal flag will also be parsed as an extension removal task and will always need to be the second argument.

blueprint -r example -user-confirmed-removal Removes example, attempts removal on -user-confirmed-removal which does not exist Does not ask for confirmation

blueprint -r example example2 -user-confirmed-removal Removes example, removes example2, attempts removal on -user-confirmed-removal which does not exist Asks for confirmation

blueprint -r example -user-confirmed-removal example2 Removes example, attempts removal on -user-confirmed-removal which does not exist, removes example2 Does not ask for confirmation

We currently do not have a standard way of parsing arguments to -remove, -install or any command for that matter.

They never tested what they wrote in general because they didn't know how to build the panel

laraproto avatar Mar 19 '25 11:03 laraproto

@laraproto :shrug:

prplwtf avatar Mar 19 '25 11:03 prplwtf

I am closing this pr the alternative is docs.docker.com/reference/cli/docker/container/exec#env

Eveeifyeve avatar Aug 01 '25 20:08 Eveeifyeve