Open zapta opened 3 years ago
Hi @zapta,
Thanks for the feedback.
I think adding a way to select the DFU interface when there are multiple (as is often the case for DfuSe) sounds like a great idea with no drawbacks.
I'm not as sure about allowing external firmware files or instructions to be passed as parameters. I agree that it would simplify life for developers that aren't as familiar with Javascript or have limited hosting resources, but as someone who only does a little bit more web development than the average embedded engineer, I don't think I have the security expertise to build it in a way that makes it difficult for someone to directly use the demo as an attack vector.
I can sympathize with the difficulty in building a polished, streamlined firmware updater interface - I would suggest taking a look at some of the forks if you want to see how other projects have adapted and refined the demo into something more user-friendly.
Thanks @devanlai, I will take a look at the forks as you suggest.
As for security, that entire thing of web pages accessing my local USB devices is strange, but that vulnerability, if it is one, was introduced by the web browsers, not by your code.
The vid=xyz URL param is very useful as it allows to construct less confusing URLs for end users.
Please add similar params for the other aspect of the DFU flashing configuraiton:
DFU: cfg=1, intf=0, alt=0, name="@Internal Flash /0x08000000/04016Kg,01064Kg,03128Kg" DFU: cfg=1, intf=0, alt=1, name="@Option Bytes /0x1FFFC000/01016 e" DFU: cfg=1, intf=0, alt=2, name="@OTP Memory /0x1FFF7800/01512 e,01016 e" DFU: cfg=1, intf=0, alt=3, name="@Device Feature/0xFFFF0000/01*004 e"
This will also allow to have on my web page multiple links, for different versions, such as stable, nightly, etc.
Having such a customizable page on github will be a great tool for makers and small developers that are either don't know javascript enough to customize or don't want to fork away to benefit from all future improvements.