search

All-Hands-AI / OpenHands

🙌 OpenHands: Code Less, Make More
https://all-hands.dev
MIT License
37.62k stars 4.25k forks source link

[Bug]: Security analyzer (invariant) is broken #5264

Open amanape opened 3 days ago

amanape commented 3 days ago

Is there an existing issue for the same bug?

Describe the bug and reproduction steps

  1. Enabling the security analyzer does not wait for user confirmation before. It always runs the command
  2. Enabling the "Invariant" option causes the app to go into an unhealthy loop. Seems to be related to the asyncio changes

OpenHands Installation

Docker command in README

OpenHands Version

0.14.2

Operating System

None

Logs, Errors, Screenshots, and Additional Context

No response

mbalunovic commented 3 days ago

I can take a look here - do you know which asyncio changes cause this issue?

amanape commented 2 days ago

I can't tell for sure. You can replicate the issue by enabling it and selecting the Invariant security analyzer. If you observe the terminal, it repeats the same asyncio error.