[bisq-network/bisq] Fix address validation for whitespace after comma (#4253)
James Cox
notifications at github.com
Sun May 10 21:59:57 UTC 2020
Typical comma separated values have a space after each comma in order to make the list more human-readable. In some areas of bisq, comma separated lists are generated this way (see #4203), but the validation of comma separated addresses was not accepting spaces after commas. This change to the regex allows optional whitespace after comma separators. A new test has been added to validate the scenario of whitespace after comma.
Fixes #4203
### Before
![image](https://user-images.githubusercontent.com/47253594/81511437-2a1b0e80-92df-11ea-991f-3883a4f0eeec.png)
![image](https://user-images.githubusercontent.com/47253594/81511447-3737fd80-92df-11ea-8116-057fb673a25c.png)
### After
![image](https://user-images.githubusercontent.com/47253594/81511502-85e59780-92df-11ea-9ffe-06cb53ec32fe.png)
![image](https://user-images.githubusercontent.com/47253594/81511496-7d8d5c80-92df-11ea-9651-8c3a36d8b492.png)
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bisq/pull/4253
-- Commit Summary --
* Fix address validation for whitespace after comma
-- File Changes --
M desktop/src/main/java/bisq/desktop/util/GUIUtil.java (2)
M desktop/src/test/java/bisq/desktop/util/GUIUtilTest.java (1)
-- Patch Links --
https://github.com/bisq-network/bisq/pull/4253.patch
https://github.com/bisq-network/bisq/pull/4253.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bisq/pull/4253
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200510/a83f5a1d/attachment.html>
More information about the bisq-github
mailing list