search

logto-io / logto

🧑‍🚀 The better identity infrastructure for developers and the open-source alternative to Auth0.
https://logto.io
Mozilla Public License 2.0
8.73k stars 437 forks source link

fix: include `tenantId` and its root param in responses #6092

Closed mostafa closed 4 months ago

mostafa commented 4 months ago

This will add tenantId back to responses, so that the generated Go client SDK doesn't complain about unknown JSON field being unmarshalled. This is the continuation of #6089 and (possibly completely) fixes #6043. The SDK works fine now, at least for a few routes I checked.

mostafa commented 4 months ago

Fill free to fix this, otherwise I'll do it in a few hours.

github-actions[bot] commented 4 months ago

COMPARE TO master

Total Size Diff :chart_with_upwards_trend: +16 Bytes

Diff by File |Name|Diff| |---|---| |packages/core/src/routes/swagger/utils/parameters.ts|:chart_with_upwards_trend: +112 Bytes| |packages/core/src/utils/zod.ts|:chart_with_upwards_trend: +96 Bytes|
mostafa commented 4 months ago

@gao-sun This seems like an intermittent failure, and can be restarted. Not 100% sure though.

gao-sun commented 4 months ago

@mostafa yes, a random thing