<p>From the OP:</p>
<blockquote>
<p>Since trades on Bisq do not happen very frequently, this cache time seems reasonable trade-off for the performance gains.</p>
</blockquote>
<p>I will just note here that the way the filecache works in the original code, data is cached indefinitely until the underlying bisq json file is written to, and then the json file is atomically read and cached.  This means that the service never serves stale data, even with caching enabled. (At least to the extent that the backend node is in sync with the p2p network).   It seems the new system does not preserve that property.</p>
<blockquote>
<p>Of course using a fully self-hosted solution is preferable to using Google Appengine, since Google would have a log of all IP addresses that access the proxy hosted on their appengine platform, and Bisq should always protect the privacy of Bisq users as much as possible.</p>
</blockquote>
<p>seriously?  To me, this is a non-starter for using Google Appengine.   Have you looked at using a reverse proxy cache like <a href="https://varnish-cache.org/" rel="nofollow">varnish</a> locally instead?</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-markets/pull/20?email_source=notifications&email_token=AJFFTNRRTYXPXHTTJXYTMGLQTGCHZA5CNFSM4JK7622KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDXLTBQ#issuecomment-552515974">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNWV5M2NGRRHOXU4OA3QTGCHZANCNFSM4JK7622A">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNWUP3F5VKTXFMFDXU3QTGCHZA5CNFSM4JK7622KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDXLTBQ.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-markets/pull/20?email_source=notifications\u0026email_token=AJFFTNRRTYXPXHTTJXYTMGLQTGCHZA5CNFSM4JK7622KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDXLTBQ#issuecomment-552515974",
"url": "https://github.com/bisq-network/bisq-markets/pull/20?email_source=notifications\u0026email_token=AJFFTNRRTYXPXHTTJXYTMGLQTGCHZA5CNFSM4JK7622KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDXLTBQ#issuecomment-552515974",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>