[Bug]: LLM-related settings fail to be saved

[avi12]

Is there an existing issue for the same bug?

• I have checked the existing issues.

Describe the bug and reproduction steps

2024-12-26.21-56-17.msedge.mp4

OpenHands Installation

Docker command in README

OpenHands Version

main

Operating System

Windows 10

Logs, Errors, Screenshots, and Additional Context

No response

[avi12]

Interestingly enough, I cannot reproduce the issue when I set in the Docker command to pull the image of 0.17

[mamoodi]

@avi12 yep thanks for bringing this up. Looks like a regression on main

[mamoodi]

@tofarr you had a PR go in recently to migrate settings?
Can you take a look?

Aside from the issue reported, seems like it now expects the security analyzer to be set as well.

[rbren]

@avi12 the docker run command changed recently. Can you share the exact command you ran?

[avi12]

docker run
  -p 3000:3000
  --env LOG_ALL_EVENTS=true
  --env SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:main-nikolaik
  --env WORKSPACE_MOUNT_PATH=C:\repositories\extensions\youtube-time-manager
  --name openhands-app
  --pull always
  --add-host host.docker.internal:host-gateway
  -v C:/repositories/extensions/youtube-time-manager:/opt/workspace_base
  -v /var/run/docker.sock:/var/run/docker.sock
  --rm
  docker.all-hands.dev/all-hands-ai/openhands:main

[enyst]

Expecting the security analyzer should be fixed now in main.

[mamoodi]

This should be resolved in main. Just one small note right now that @amanape will work on fixing in a follow up.
The API KEY will show empty in the UI even if you set it. It works though. Just shows empty in UI.