[bisq-network/bisq-pricenode] Don't deliver price data if request from source fails (#13)
Manfred Karrer
notifications at github.com
Fri Jun 15 08:12:53 UTC 2018
We had recently a case where one price node was failing due an unknown error [1] and clients still got delivered data from that node but the data was outdated (1000 USD price difference to other nodes). We have a timestamp for the delivery to the Bisq node to ignore outdated prices, but the age of the source data is not checked.
So we should add some check at the requests to the price providers (BTC Average,...) to detect if we don't get fresh data and if the requested data are not recent the time stamp for the requests from Bisq nodes should not be updated to reflect the real age if the price data.
I am not 100% sure how it is/was implemented but as the nodes did not detect the outdated prices I assume that the timestamp is not set at the price providers side.
[1] That was logged int eh error.log file:
Exception in thread “Timer-5” java.lang.StackOverflowError
at java.lang.reflect.InvocationTargetException.<init>(InvocationTargetException.java:72)
at sun.reflect.GeneratedMethodAccessor73.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at ch.qos.logback.classic.spi.ThrowableProxy.<init>(ThrowableProxy.java:66)
at ch.qos.logback.classic.spi.ThrowableProxy.<init>(ThrowableProxy.java:60)
at ch.qos.logback.classic.spi.ThrowableProxy.<init>(ThrowableProxy.java:72)
--
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-pricenode/issues/13
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20180615/964b9021/attachment-0001.html>
More information about the bisq-github
mailing list