search

louislam / dockge

A fancy, easy-to-use and reactive self-hosted docker compose.yaml stack-oriented manager
https://dockge.kuma.pet
MIT License
10.37k stars 290 forks source link

auto removes " " in compose file while deploying the stack #118

Closed shreehari1 closed 7 months ago

shreehari1 commented 7 months ago

have a application which requires "" while defining volumes. it removes the "/data:/dada" -> /data:/data and container fails to deploy

louislam commented 7 months ago

Can you show the example of that path?

shreehari1 commented 7 months ago

`volumes:

volumes:

louislam commented 7 months ago

Cannot reproduce, my stack is able to start without any errors using your volumes

What is the error msg?

shreehari1 commented 7 months ago

I did some digging from my side too. looks like there is some issue with the image i was trying. I am not facing it on there images. Loving the project, waiting for more things coming up. all the best.

zebulun78 commented 6 months ago

This is still a problem. I have an image that requires the quotes to remain, and dockge is stripping the quotes from my compose file...

zebulun78 commented 6 months ago

The quotes get stripped when you go to edit the compose file, even though it saves them properly the first time

radh21301 commented 3 months ago

I have the exact same issue as @zebulun78

bverkron commented 2 months ago

I have the same issue. Why was this closed? @louislam I have a working example if you need one. I (and perhaps others that have commented) don't get errors when starting the stack but it is still incorrect for dockge to remove quotes. Quotes are often required to encapsulate strings with special characters like secrete keys, etc. Stripping the quotes can break things inside the container at runtime or cause unexpected behaviour, even if it starts correctly.

This is actually a show stopper for certain stacks where I must be able to quote certain values or it breaks functionality due to special characters.