iSpy icon indicating copy to clipboard operation
iSpy copied to clipboard

[Agent] Umlaute in camera names not properly encoded in webview

Open SaculRennorb opened this issue 8 months ago • 5 comments

Using any umlaut, e.g. ü in the name of a device will properly show that name in the server console, but render as the replacement character in the webview.

SaculRennorb avatar Dec 06 '23 18:12 SaculRennorb

hello whereabouts in the web view? what web browser are you using? image image image

ispysoftware avatar Dec 07 '23 02:12 ispysoftware

Using FF version 120.0 (64-bit)

Interestingly I'm unable to reproduce this now, but it was in the title of the edit device (?) modal. grafik

I cannot figure out what I'm doing different now, but I'm certain it wasn't encoded properly a few hours ago. I cannot remember if reloading the page changed anything, but it was happening multiple times before I reported it.

Unfortunately this is one of those 'largely irrelevant' things I want paying much attention to after the fact.

SaculRennorb avatar Dec 07 '23 02:12 SaculRennorb

Ha grafik grafik

grid view -> new device -> just selected a dummy one and put umlaute in the name -> ok -> now its broken everywhere

SaculRennorb avatar Dec 07 '23 02:12 SaculRennorb

I think this is probably more a bug in firefox - have you tried it in chrome - unable to reproduce it there

ispysoftware avatar Dec 07 '23 02:12 ispysoftware

Hm, you're right. I also cannot repro this using Chrome. From my experience its usually Chrome doing something that's not to spec but lib authors relying on that, and FF just not doing that off spec thing. But tbh I have no idea what would be causing it in this case, since any apis for dynamically creating nodes have been out for quite a while and should be well tested.

I recently had something similar happen where if you don't specify the encoding in the meta tag of the html header Chrome assumes its CP 1252 (i think) while FF just assumes the much more reasonable UTF8, which would break special ascii characters in chrome. No idea if this is relevant here.

But I guess that's it then, not really a priority to work on this but I guess its nice to track the issue.

SaculRennorb avatar Dec 07 '23 02:12 SaculRennorb