Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove non working legacy pools #145

Merged
merged 1 commit into from
Aug 3, 2021
Merged

Conversation

degeri
Copy link
Member

@degeri degeri commented Jun 17, 2021

https://stakey.net => redirects to vspd
https://dcr.farm => redirects to vspd
https://testnet.decredvoting.com => down
https://dcrstakedinner.com => bad ssl and 502

@xaur
Copy link

xaur commented Jun 21, 2021

dcr.farm indeed redirects to vsp.dcr.farm, even requests to API endpoints. As I wrote in #146 this may be inconvenient.

stakey.net shows the new vspd page but its legacy status endpoint still works and reports 86 tickets as of writing. Their support post says the legacy UI was removed but wallets and API will stay up.

I'm not entirely sure we should remove it from the listing. On the one hand the admin has clearly communicated to migrate to vspd, the API says it is Closed, and we should not advertise it in any way now. But if we remove it from the listing, the vsp page will no longer show its live ticket count and "updated X minutes ago", which is a comforting message that it is still up and voting the remaining tickets.

@degeri
Copy link
Member Author

degeri commented Jun 22, 2021

stakey.net is throwing an error for users in decrediton. It is better to remove it.

@xaur
Copy link

xaur commented Jun 23, 2021

Oh I forgot that this API is also consumed by Decrediton. I agree it should be the priority use case and so any malfunctioning VSPs should be removed.

Perhaps we need a separate API for monitoring all VSPs.

@xaur
Copy link

xaur commented Jul 13, 2021

I can access https://dcrstakedinner.com/ now

@jholdstock jholdstock merged commit 55cbd75 into decred:master Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants