Skip to content

Conversation

@ldez
Copy link
Member

@ldez ldez commented Mar 20, 2025

The type of ReturnInfo doesn't seem "stable".

allinkl doesn't have a real API schema: https://kasapi.kasserver.com/soap/wsdl/KasApi.wsdl

<message name="KasApiAntwort">
<part name="Result" type="xsd:anyType"/>
</message>
2025/03/20 12:54:56 [WARN] [*.example.com] acme: cleaning up failed: allinkl: response struct decode: decoding failed due to the following error(s):

'Response.ReturnInfo' expected type '%!s(bool=false)', got unconvertible type '""', value: '""'

Related to #2489

@ldez ldez added this to the unreleased milestone Mar 20, 2025
@ldez ldez requested a review from dmke March 20, 2025 15:02
Copy link
Member

@dmke dmke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

allinkl doesn't have a real API schema

Oh, the schema is real. It's just utterly useless.

@ldez
Copy link
Member Author

ldez commented Mar 20, 2025

Yes it's a real decorative API schema 😄

@ldez ldez enabled auto-merge (squash) March 20, 2025 15:11
@ldez ldez merged commit 0fae2f0 into go-acme:master Mar 20, 2025
7 checks passed
@ldez ldez deleted the fix/allinkl-response branch March 20, 2025 15:27
@ldez ldez modified the milestones: unreleased, v4.23 Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants