<p></p>
<blockquote>
<p>I installed your PR branch and it crashes on startup, you owe me a thousand bucks <g-emoji class="g-emoji" alias="grin" fallback-src="https://github.githubassets.com/images/icons/emoji/unicode/1f601.png">😁</g-emoji></p>
</blockquote>
<p>It failed cause BitcoinAverage didn't find the API keys. I kind of assumed the PR would be used to upgrade a current pricenode, or at least using the same environment setup (e.g. including the BA keys). But of course, if it's used without the API keys, it will fail.</p>
<p>So alright, kind of a gray area -- I wasn't clear enough on this in the bounty rules. But I guess that's on me, lesson learned :)</p>
<p>So sure, you get the bounty :) We'll settle this once the compensation is out.</p>
<blockquote>
<p>Can you just delete the Bitcoin Average provider?</p>
</blockquote>
<p>Will do, commit coming soon.</p>
<blockquote>
<p>But obviously a 404 error should not cause the pricenode to crash, this is also why CMC returning 410 a while ago caused our pricenodes to crash. It seems this critical exception is not handled?</p>
</blockquote>
<p>Yea and no. This failure is specific to the BA setup, which expects to find an API key, otherwise it won't start up.</p>
<p>If the pricenode is running (started fine), but then some API endpoint returns errors, that should be handled (see Test 5).</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bisq/pull/4315#issuecomment-668736594">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNTS3YQTDSSLWQ5S7TLR7BCZRANCNFSM4N75VIBA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNWYOTB3GZVLA5MAFV3R7BCZRA5CNFSM4N75VIBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOE7OBYUQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/4315#issuecomment-668736594",
"url": "https://github.com/bisq-network/bisq/pull/4315#issuecomment-668736594",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>