[bisq-network/bisq] Fix localization issue when accessing directories and JAR resource paths (PR #6148)
jmacxx
notifications at github.com
Wed Apr 13 01:39:45 CEST 2022
Fixes #5722
Fixes #6144
Two problems were encountered under Windows with the locale and language set to Turkish:
1. JAR-contained resource `btc_mainnet.seednodes` could not be located because the "i" was translated to "ı" resulting in no seednodes being known.
2. the Bisq data directory `btc_mainnet` was created as `btc_maınnet` and as a result Tor would not start.
The solution in both cases is to tell `.toLowerCase()` to use Locale.ENGLISH because directory names such as btc_mainnet are locale-insensitive.
Ref:
https://stackoverflow.com/questions/10336730/which-locale-should-i-specify-when-i-call-stringtolowercase
https://stackoverflow.com/questions/11063102/using-locales-with-javas-tolowercase-and-touppercase#11063161
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/6148
-- Commit Summary --
* Use Locale.ENGLISH when accessing directories and JAR resource paths.
-- File Changes --
M common/src/main/java/bisq/common/config/Config.java (3)
M core/src/main/java/bisq/core/network/p2p/seed/DefaultSeedNodeRepository.java (3)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/6148.patch
https://github.com/bisq-network/bisq/pull/6148.diff
--
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/6148
You are receiving this because you are subscribed to this thread.
Message ID: <bisq-network/bisq/pull/6148 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20220412/4e08650b/attachment.htm>
More information about the bisq-github
mailing list