diff options
| author |  Vyr Cossont <vyr@noreply.codeberg.org> | 2025-11-11 19:39:09 +0100 |
|---|---|---|
| committer |  tobi <tobi.smethurst@protonmail.com> | 2025-11-17 14:15:04 +0100 |
| commit | 0c7b069c4a09f01a3a6c59bd7b66f096f8dcec40 (patch) | |
| tree | e5306d5ebad1e5a6efe5393bfafde95f5b94b237 /internal/api/client/trends | |
| parent | [performance] remove hard reliance on .Cached field to indicate whether media... (diff) | |
| download | gotosocial-0c7b069c4a09f01a3a6c59bd7b66f096f8dcec40.tar.xz | |
[chore] Apply schema for OpenAPI errors (#4511)
Adds an error struct to Swagger covering normal and OAuth errors for client API endpoints. Now clients using Swagger codegen can handle errors.
Specifically, if you _don't_ have typed errors in your Swagger IDL and you're using `go-swagger` to generate your API client, the error message is discarded and you are only told the status code. With this change in place, clients like `slurp` can tell the user that they tried to upload an emoji that was too big or whatever.
Reviewed-on: https://codeberg.org/superseriousbusiness/gotosocial/pulls/4511
Reviewed-by: tobi <kipvandenbos@noreply.codeberg.org>
Co-authored-by: Vyr Cossont <vyr@noreply.codeberg.org>
Co-committed-by: Vyr Cossont <vyr@noreply.codeberg.org>
Diffstat (limited to 'internal/api/client/trends')
| -rw-r--r-- | internal/api/client/trends/links.go | 2 | ||||
| -rw-r--r-- | internal/api/client/trends/statuses.go | 2 | ||||
| -rw-r--r-- | internal/api/client/trends/tags.go | 2 |
3 files changed, 6 insertions, 0 deletions
diff --git a/internal/api/client/trends/links.go b/internal/api/client/trends/links.go index 72444538e..3a8b77dcb 100644 --- a/internal/api/client/trends/links.go +++ b/internal/api/client/trends/links.go @@ -46,6 +46,8 @@ import ( // type: object // maxItems: 0 // '406': +// schema: +// "$ref": "#/definitions/error" // description: not acceptable func (m *Module) LinksGETHandler(c *gin.Context) { if _, err := apiutil.NegotiateAccept(c, apiutil.JSONAcceptHeaders...); err != nil { diff --git a/internal/api/client/trends/statuses.go b/internal/api/client/trends/statuses.go index f8282095e..9e79e5072 100644 --- a/internal/api/client/trends/statuses.go +++ b/internal/api/client/trends/statuses.go @@ -46,6 +46,8 @@ import ( // type: object // maxItems: 0 // '406': +// schema: +// "$ref": "#/definitions/error" // description: not acceptable func (m *Module) StatusesGETHandler(c *gin.Context) { if _, err := apiutil.NegotiateAccept(c, apiutil.JSONAcceptHeaders...); err != nil { diff --git a/internal/api/client/trends/tags.go b/internal/api/client/trends/tags.go index a309082bf..aa2fa81ef 100644 --- a/internal/api/client/trends/tags.go +++ b/internal/api/client/trends/tags.go @@ -46,6 +46,8 @@ import ( // type: object // maxItems: 0 // '406': +// schema: +// "$ref": "#/definitions/error" // description: not acceptable func (m *Module) TagsGETHandler(c *gin.Context) { if _, err := apiutil.NegotiateAccept(c, apiutil.JSONAcceptHeaders...); err != nil { |