Add ability to set the upload_firmware timeout in cli #77

Activity

1. martonilles commented on Jul 17, 2026

   

   martonilles

   on Jul 17, 2026

   Contributor

   More actions

   It is already possible to override the timeout default 60sec, by providing a timeout parameter:

       def upload_firmware(
           self,
           metadata: m.FirmwareMetadata,
           path: Path | None,
           *,
           sbom_path: Path | None = None,
           enable_monitoring: bool,
           timeout=60,
       ):
   
2. martonilles

   closed this as completedon Jul 17, 2026
3. CMon commented on Aug 12, 2026

   

   CMon

   on Aug 12, 2026

   Author

   More actions

   Yes I found it inside your class (thats where i got the 60s from) but not exposed to the cli itself, since I use your pip package in my CI's there the def upload_firmware from : onekey_client/cli/firmware_upload.py#L42 will be used, which does not allow setting the timeout
4. martonilles commented on Aug 14, 2026

   

   martonilles

   on Aug 14, 2026

   Contributor

   More actions

   Sorry, it was not clear you were referring to the cli
5. martonilles

   reopened this on Aug 14, 2026
6. martonilles

   changed the title [-]Add ability to set the upload_firmware timeout[/-] [+]Add ability to set the upload_firmware timeout in cli[/+] on Aug 14, 2026
7. martonilles commented on Aug 26, 2026

   

   martonilles

   on Aug 26, 2026

   Contributor

   More actions

   Done in #79
8. martonilles

   closed this as completedon Aug 26, 2026