[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