inventree / inventree-app

InvenTree mobile app
https://docs.inventree.org/en/latest/app/app/
MIT License
57 stars 48 forks source link

[FR] Scanner wedge mode - extend to all APP pages #440

Open 42CrMo4 opened 1 year ago

42CrMo4 commented 1 year ago

I would like to propose extending the scanner wedge mode introduced in #437 to work in the APP without having to go into explicit scanner mode. This would improve usability and speed of operation.

matmair commented 1 year ago

Wouldn't that mean that the inventree app would always be streaming all keyboard inputs to the server? Seems like a security/privacy problem

SchrodingersGat commented 1 year ago

@matmair it is a good thought, worth considering:

Wouldn't that mean that the inventree app would always be streaming all keyboard inputs to the server?

Only on the home screen, which is not insignificant

It would be sending data to the default barcode "scan" API endpoint only. Still, if connected to a keyboard which is streaming in a lot of data, could potentially overwhelm the server

42CrMo4 commented 1 year ago

That is a valid point. I though the Lib is only streaming the data in case of a potential barcode (ASCII and fast enough not be human, according to the readme). But if that is not the case it would be not a clean way.