feat: add npm registry config for openclaw by zhengkunwang223 · Pull Request #12300 · 1Panel-dev/1Panel
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't make other settings depend on a running container
UpdateOtherConfig now writes openclaw.json and then immediately returns docker exec ... npm set registry. Docker documents exec as operating on a running container, so if the OpenClaw container is stopped, this endpoint now fails after the file has already been updated. In that state users can't reliably change timezone/browser settings anymore, and the API reports an error even though it has partially applied the change on disk.
Useful? React with 👍 / 👎.