[bisq-network/bisq] Fix btcnode validation regex to handle multiple FQDNs, add tests (#3964)

wiz notifications at github.com
Tue Feb 11 12:17:27 UTC 2020


@devinbileck's PR #3961 fixed almost everything except one case for btcnode configuration, multiple FQDNs:
```
foo.example.com:8333,bar.example.com:8333
```

This PR fixes the FQDN regex to handle multiple cases and adds new testcases to verify
You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bisq/pull/3964

-- Commit Summary --

  * Fix btcnode validation regex to handle multiple FQDNs, add tests

-- File Changes --

    M desktop/src/main/java/bisq/desktop/util/GUIUtil.java (2)
    M desktop/src/test/java/bisq/desktop/util/GUIUtilTest.java (6)

-- Patch Links --

https://github.com/bisq-network/bisq/pull/3964.patch
https://github.com/bisq-network/bisq/pull/3964.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/3964
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20200211/7bee57d4/attachment-0001.html>


More information about the bisq-github mailing list