Commit Graph

11648 Commits

Author SHA1 Message Date
sqrrm
f8a8904cd6
Merge pull request #4431 from ghubstan/3-remove-dupllicated-grpcstubs
Remove duplicated grpc stub creation logic
2020-08-31 16:34:53 +02:00
sqrrm
00cc9a1f7d
Merge pull request #4430 from ghubstan/2-mv-bats-test
Move bats based mainnet test from :cli to :apitest
2020-08-31 16:24:12 +02:00
sqrrm
de56c1efb9
Merge pull request #4429 from ghubstan/1-add-setup-scaffold-param
Fix setUpScaffold() signature
2020-08-31 16:21:27 +02:00
sqrrm
944825e24b
Close other tab when opening signing tab 2020-08-31 16:18:03 +02:00
sqrrm
51908fcb2a
Use mediation cases to popupate arbitrator signing in dev mode 2020-08-31 15:26:44 +02:00
sqrrm
91e30de637
Make a separate signing tab
Helps dev work to have a separate signing area without need for legacy
arbitrator registration. Legacy arbitrator key is still needed to sign
accounts. Account information is also needed.
2020-08-31 14:40:57 +02:00
chimp1984
6c60e1739d
Add support for user name for Revolut accounts
If a user has an existing account with phone number or email as
account ID we show a popup at startup where we require that he sets the
user name. This popup has no close button so he is forced to enter a
value. If there are multiple account multiple popups will be shown.

To not break signed accounts we keep accountId as internal id used for signing.
Old accounts get a popup to add the new required field userName but accountId is
left unchanged. Newly created accounts fill accountId with the value of userName.
In the UI we only use userName.

Input validation does only check for length (5-100 chars). Not sure what
are the requirements at Revolut. Can be changes easily if anyone gets
the specs.
2020-08-30 12:58:31 -05:00
chimp1984
49d212e654
Fix tests.
I don't know why the tests failed as I just added an overloaded method
and it should not have any impact. There is also one exception which
makes it even more obscure. I guess its some test framework issue.

See comment at the exceptional handling
// If we remove the last argument (isNull()) tests fail. No idea why as the broadcast method has an
/ overloaded method with nullable listener. Seems a testframework issue as it should not matter if the
// method with listener is called with null argument or the other method with no listener. We removed the
// null value from all other calls but here we can't as it breaks the test.
2020-08-30 10:37:46 -05:00
chimp1984
8d13ff8856
Add comment about size 2020-08-30 10:02:19 -05:00
chimp1984
0aa97d39ab
Merge remote-tracking branch 'origin/bundle-msg-at-broadcast' into bundle-msg-at-broadcast 2020-08-30 07:28:02 -05:00
chimp1984
6accd58fc9
Merge branch 'master_upstream' into bundle-msg-at-broadcast
# Conflicts:
#	p2p/src/main/java/bisq/network/p2p/P2PService.java
2020-08-30 07:27:15 -05:00
Christoph Atteneder
51e66d5763
Merge pull request #4447 from wiz/tweak-bitcoin-conf-timeout
Tweak seednode bitcoin.conf, remove timeout, separate testnet block
2020-08-30 14:25:29 +02:00
chimp1984
ebecd20949
Merge branch 'master_upstream' into add-check-for-option-trade 2020-08-29 20:00:07 -05:00
chimp1984
bedcd9b10e
Add option trade detection message to result message if case was
marked as option trade.

Add repeated calls to price provider it no data available (can happen at
startup).

Show reason in dispute summary.
2020-08-29 19:58:51 -05:00
chimp1984
f2e8ce582e
Remove timeStampMap as it is never read
Add hasPrices method
Remove final keywords
2020-08-29 19:29:59 -05:00
chimp1984
beeac10a99
Change GMT0 to UTC and add UTC time to log at shut down 2020-08-29 19:29:08 -05:00
chimp1984
08632e13ee
Add message after dispute opening to show if trade might be an option trade 2020-08-29 18:00:36 -05:00
chimp1984
bf2ca1fc0c
Add BTC prefix or postfix to Price.toFriendlyString method 2020-08-29 14:07:59 -05:00
chimp1984
01482fb8e6
Refactoring
Separate validation from setting key
2020-08-29 13:21:19 -05:00
chimp1984
b0eea78a03
Remove unused properties (was added again from a merge failure) 2020-08-29 13:14:18 -05:00
chimp1984
366fdb3a09
Merge remote-tracking branch 'origin/bundle-msg-at-broadcast' into bundle-msg-at-broadcast
# Conflicts:
#	p2p/src/main/java/bisq/network/p2p/P2PService.java
2020-08-29 13:13:31 -05:00
chimp1984
be78770369
Merge branch 'master_upstream' into bundle-msg-at-broadcast
# Conflicts:
#	p2p/src/main/java/bisq/network/p2p/P2PService.java
2020-08-29 13:04:42 -05:00
Christoph Atteneder
ecb8db3a4e
Merge pull request #4440 from jmacxx/fix_low_bsq_offer
Improve warning message regarding insufficient BSQ
2020-08-29 20:03:03 +02:00
Christoph Atteneder
835c511cf5
Merge pull request #4448 from wiz/revert-4398-header-tweaks
Revert "Header and other visual tweaks"
2020-08-29 20:01:15 +02:00
Christoph Atteneder
589266966c
Merge pull request #4450 from chimp1984/fix-getMinRefundAtMediatedDispute
Fix null pointer in case if selectedMarketPrice is null
2020-08-29 19:57:55 +02:00
chimp1984
7a4fc3d6b7
Merge branch 'master_upstream' into refactor-filter 2020-08-29 12:56:29 -05:00
chimp1984
9e4e800cb6
Fix tests 2020-08-29 12:53:35 -05:00
chimp1984
ef11e2ad89
Fix comment 2020-08-29 12:29:07 -05:00
chimp1984
8253957cb2
Fix null pointer in case if selectedMarketPrice is null 2020-08-29 12:19:36 -05:00
sqrrm
38f9d251ae
Merge pull request #4435 from chimp1984/improve-network-statistics
Add more detailed network statistics
2020-08-29 19:01:43 +02:00
sqrrm
591d68a6fd
Merge pull request #4445 from chimp1984/restart-seednodes-deterministically
Use the index of a seed node address in the list of nodes at the
2020-08-29 15:20:03 +02:00
sqrrm
79a26aa95d
Merge pull request #4446 from chimp1984/add-report-button-for-dispute-views
Improve dispute views
2020-08-29 14:18:21 +02:00
wiz
997a2a5e55
Revert "Header and other visual tweaks" 2020-08-29 18:12:23 +09:00
chimp1984
b5c8bae1fa
Enable check for lock time on mainnet and add a 3 block tolerance. 2020-08-28 22:16:04 -05:00
chimp1984
f52beadc47
Improve dispute views
Add re-open dispute button to mediation views (not refund agent as
there it might be dangerous if he would close 2 times and pay out twice.

If a dispute was reopened and a message was sent afterwards the receiver
will auto-reopen the dispute as well and the UI shows the number
indicator.

Add search field to traders dispute views.

Fix DisputeMsgEvents which was handling only legacy arbitration cases
and now uses mediation and refund agent. Used to update state in case
the UI is not selected.

Add "send private notification" button for mediators and refund agents.

Add report button for mediators and refund agents
Generates report text of all disputes with more detailed information
which should be used for reporting.

Add "Dump text of all disputes" button for getting a text representation
of all disputes including the messages for mediators and refund agents.
This is not used for report but might be useful for search, etc.

Do not show legacy arbitration tabs (for traders as well for
arbitrators) if no cases available.

Add 5 new enum entries for Reason for dispute:
OPTION_TRADE
SELLER_NOT_RESPONDING
WRONG_SENDER_ACCOUNT,
TRADE_ALREADY_SETTLED,
PEER_WAS_LATE

Hide NO_REPLY, SCAM and PROTOCOL_VIOLATION in UI (still in code if we still
need it and re-activate them).
2020-08-28 22:15:11 -05:00
wiz
243bc19ae2
Tweak seednode bitcoin.conf, remove timeout, separate testnet block 2020-08-29 08:54:05 +09:00
chimp1984
97ec45c173
Merge branch 'reenable-chat-on-dispute-reopen' into add-report-button-for-dispute-views 2020-08-28 14:23:36 -05:00
chimp1984
a148850685
Merge branch 'master_upstream' into reenable-chat-on-dispute-reopen 2020-08-28 14:11:59 -05:00
chimp1984
fa9f799186
Improve dispute views
Add report button:
Generates report text of all disputes with more detailed information
which should be used for reporting.

Add "Dump text of all disputes" button for getting a text representation
of all disputes including the messages. This is not used for report but
might be useful for search, etc.

Do not show legacy arbitration tabs (for traders as well for
arbitrators) if no cases available.

Add 3 new enum entries for Reason for dispute:
OPTION_TRADE
SELLER_NOT_RESPONDING
WRONG_SENDER_ACCOUNT
Hide NO_REPLY and PROTOCOL_VIOLATION in UI (still in code if we still
need it and re-activate them)
2020-08-28 13:48:28 -05:00
sqrrm
d1a1cc52fe
Merge pull request #4410 from chimp1984/optimize-shutdown-routines
Optimize application shutdown
2020-08-28 17:17:01 +02:00
chimp1984
c535ebbff0
Merge branch 'improve-network-statistics' into bundle-msg-at-broadcast 2020-08-28 09:48:59 -05:00
chimp1984
5ac1f6e05c
Merge branch 'master_upstream' into improve-network-statistics 2020-08-28 09:48:14 -05:00
chimp1984
5f7cfd2dc8
Improve logging of initial data requests 2020-08-28 09:39:33 -05:00
Christoph Atteneder
3599355e4a
Merge pull request #4442 from cd2357/patch-1
Exclude spot price provider tests by default
2020-08-28 14:57:14 +02:00
chimp1984
c218b28755
Satistfy codacy 2020-08-28 07:39:23 -05:00
chimp1984
fa5a58d41a
Use the index of a seed node address in the list of nodes at the
repository to determine the hour to restart.
2020-08-28 02:30:45 -05:00
chimp1984
28a665e02d
Satisfy annoying Codacy bot 2020-08-27 21:29:30 -05:00
chimp1984
2f291a6acb
Cleanup Filter value object. Remove nullables from old versions. Keep signatureAsBase64 nullable as it is set null at creation time and set later after signing. We use filter protobuf data as input for signature so we have to set signature to null and apply the signature afterwards. To keep object immutable we clone it with static clone method. Similar we handle signature verification. We clone the filter with a null value for the signature and do the validation with that cloned object.
Use now the complete object data for signature creation. We use the protobuf data for creating the signature with DER encoding. We restict ourself more regarding backward compatibility but I think it is not a big problem.
If a new field is added to Filter and deployed the maintainer needs to publishes a new filter object. By using the new version he cannot remove the old filter from the network as the protobuf data is different and sig verification on the P2P datastorage level will fail. This is intended to keep the old filter alive for some time to support not updated users. We do not remove invalid filters anymore from our local storage to enable seed nodes to support also old filter objects.
For not updated users the new filter will fail at sig verification because the protobuf data is different. So they ignore the new filter and still use the old filter. For updated users the old filter will fail and the new filter is accepted. As it has a newer date it would also replace the old filter anyway. If the maintainer wants to delete the old filter from the network he can use the old app version and remove the filter. It is recommended to keep a copy of the data directory before the update so that the removal of the older filter is possible.

Refacorings:
- Rename isPeersPaymentAccountDataAreBanned to arePeersPaymentAccountDataBanned
- Rename isSignerPubKeyBanned to isWitnessSignerPubKeyBanned
2020-08-27 21:23:02 -05:00
chimp1984
0e8704e74f
Remove dev logs, remove unread property 2020-08-27 16:50:45 -05:00
chimp1984
21ff2df232
Remove dev logs 2020-08-27 16:42:55 -05:00