fix: bump SDK v0.16.2 (empty balancer/ssl list) + tolerate isCreateEnable - #79
Merged
Merged
Conversation
…cer config v0.16.2 fixes the empty balancer/cloud-ssl list decode (dogfooding an account with no balancers or certs), so 'sweb balancer list' and 'sweb ssl list' work again. 'sweb balancer config' also failed entirely because isCreateEnable answers -32602 for some account types (the spec documents it param-less, but the live API rejects every param shape). The catalog is still useful, so report create-availability as 'unknown' on error instead of failing — same tolerance as 'domains price'. Assisted-By: Claude <noreply@anthropic.com> Signed-off-by: sanchpet <petrov@sanch.pet>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
sweb-go-sdkto v0.16.2, which fixes the emptybalancer list/ssl listdecode — the released CLI (v0.18.0) errored on an account with no balancers or cloud certs.sweb balancer configtolerant ofisCreateEnablefailing: it answers-32602for some account types (the OpenRPC spec documents it param-less, but the live API rejects every param shape), so the whole command failed. The plans/protocols catalog is still useful, so create-availability is now reported asunknownon error instead of aborting — the same per-call tolerance asdomains price.Why
Found by dogfooding a sweep of the live account:
balancer list,balancer config, andssl listall errored on the released CLI. The first two/last are now clean.Verification
Verified live against the account (empty balancers/certs):
balancer listandssl listrender,balancer configshows the catalog withCREATE AVAILABLE unknown.mise run cigreen.