<blockquote>
<p>I noticed when generating a new address and restarting (without open trades/offers), both addresses are still in use</p>
</blockquote>
<p>hm I have not encountered this case. Will look into it.</p>
<blockquote>
<p>it listed a seednode twice in the connected peers list</p>
</blockquote>
<p>I have not encountered this case. Will look into it.</p>
<blockquote>
<p>I am not sure if it failed to clean it up initially or if it needs an additional check on startup for cleanup?</p>
</blockquote>
<p>if cleaning unused hidden services on startup would be easy, I would have done it right away. The issue is the async way of services getting started - by the time Bisq can assess which HS to keep, all HSs are already up and running (at least most of the time). So no luck here.</p>
<blockquote>
<p>Additionally, I still feel the GUI should explicitly indicate all onion addresses in use...</p>
</blockquote>
<p>we could just add a field "legacy onion addresses" with a list of onion addresses still in use, but I am not sure if that can easily be understood by the average Joe. As for the connections, the legacy HS should only receive connections from peers which know the onion address so I see no need for adding an additional connection listing. All in all, I would suggest to get the GUI as is into production and see if and how people actually use it.</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/3044?email_source=notifications&email_token=AJFFTNQVTIHQA4VSHFAPHGTQVT6SBA5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFFTO6A#issuecomment-558577528">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNR5GZORT2CRCA3HVK3QVT6SBANCNFSM4IIPXYQQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNS3F7YLGH4AABLQ3UTQVT6SBA5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFFTO6A.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/3044?email_source=notifications\u0026email_token=AJFFTNQVTIHQA4VSHFAPHGTQVT6SBA5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFFTO6A#issuecomment-558577528",
"url": "https://github.com/bisq-network/bisq/pull/3044?email_source=notifications\u0026email_token=AJFFTNQVTIHQA4VSHFAPHGTQVT6SBA5CNFSM4IIPXYQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFFTO6A#issuecomment-558577528",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>