nimi-python icon indicating copy to clipboard operation
nimi-python copied to clipboard

windows-1251 is hardcoded in our APIs as the encoding to use

Open ni-jfitzger opened this issue 3 years ago • 4 comments

Description of issue

'windows-1251' is hardcoded in our APIs as the encoding to use. We support Linux, as well, so this definitely doesn't seem right.

System report

None

Steps to reproduce issue

Just look at the code.

ni-jfitzger avatar Sep 19 '22 17:09 ni-jfitzger

Unfortunately, @marcoskirsch, if we discussed what the correct solution is, I've already forgotten it.

ni-jfitzger avatar Sep 19 '22 17:09 ni-jfitzger

We need to figure out what the correct encoding in Linux is for our driver runtimes, then select the right encoding based on the platform we're running on.

I think a good way to test is to use two devices whose aliases have non-ascii characters, and confirm we can create sessions and make routes between them.

marcoskirsch avatar Sep 19 '22 17:09 marcoskirsch

Once we get this working right, we should ensure there is unit-test coverage.

marcoskirsch avatar Sep 19 '22 17:09 marcoskirsch

Note that windows-1251 is for languages that use Cyrillic script, such as Russian, Ukrainian, and Belarusian. You probably meant to hardcode windows-1252, which is roughly equivalent to Latin-1.

bkeryan avatar Mar 02 '23 15:03 bkeryan