Camera ships with no password, not 12345 - #132
Conversation
Current firmware gives root no password at all. Until one is set the camera streams nothing, and the setup page it serves is the only thing it will answer, so there is nothing to sign in with beforehand. The flashing instructions still told people to sign in as root/12345 and change it afterwards, which describes firmware nobody is flashing any more. Rewritten for en, ru and zh -- the three locales this repository still carries. The text now says what the camera actually does: it has no password yet, it streams nothing until it has one, it asks for one before anything else, and that password is root's, shared with ssh, UART, RTSP and ONVIF. The Russian phrasing deliberately avoids a colon after the </b> tag: these values are unquoted YAML scalars, and a colon followed by a space inside one turns the rest of the line into a mapping and breaks the file at load time.
PR Summary by QodoCorrect default-password guidance across supported locales
AI Description
Diagram
High-Level Assessment
Files changed (3)
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can type 'qodo, fix this' on a finding and the fix lands right on your PR |
Review on the wiki side (OpenIPC/wiki#500) caught this in the same sentence. The root password is what the web interface, ssh, UART and RTSP check, and ONVIF accepts it from clients sending HTTP Basic or WSSE PasswordText -- but not from ones authenticating with a digest. A digest is computed over the cleartext password and the system database stores only a one-way hash, so those clients need onvif.password set separately, in cleartext. It is the common case rather than a corner: PasswordDigest is what most ONVIF clients try first.
|
Corrected in the latest commit: the sentence claimed the root password is what ONVIF checks, which is only half true. ONVIF accepts it from clients sending HTTP Basic or WSSE PasswordText, but not from ones that authenticate with a digest — a digest is computed over the cleartext password, and the system database stores only a one-way hash, so there is nothing to compute it from. Those clients need Caught by review on OpenIPC/wiki#500, which carried the same sentence. |
Current firmware gives root no password at all. Until one is set the camera streams nothing, and the setup page it serves is the only thing it will answer — so there is nothing to sign in with beforehand.
The post-flash instructions still said to sign in as
root/12345and change it afterwards, which describes firmware nobody is flashing any more.What it says now
Rewritten for en, ru and zh — the three locales this repository still carries. (The de/es/fa/fr/it/pl/pt files were dropped upstream; I'd initially written all ten before refetching and finding them gone.)
One thing worth knowing for future edits here
These
info_htmlvalues are unquoted YAML scalars. A colon followed by a space anywhere inside one turns the rest of the line into a mapping and the file fails to load. My first Russian draft read<b>root</b>: его же используют…and brokeru.ymlexactly that way; it now uses a comma instead.Verified by parsing all 15 files in
config/locales/withyaml.safe_loadand reading the three changed values back out of the parsed tree, rather than eyeballing the diff.