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

Fjern ETag fra samtlige request body i swagger - generell swagger-cleanup #185

Closed
Tracked by #29
MagnusSandgren opened this issue Sep 26, 2023 · 1 comment · Fixed by #246
Closed
Tracked by #29

Fjern ETag fra samtlige request body i swagger - generell swagger-cleanup #185

MagnusSandgren opened this issue Sep 26, 2023 · 1 comment · Fixed by #246
Assignees
Labels
bug Something isn't working

Comments

@MagnusSandgren
Copy link
Collaborator

MagnusSandgren commented Sep 26, 2023

ETag eksponeres som et felt på body på samtlige command endepunkt (skrive endepunkt). Denne skal kun eksponeres som en header (if-match), og skal dermed bort fra request body.

Image

@MagnusSandgren MagnusSandgren converted this from a draft issue Sep 26, 2023
@elsand elsand added the bug Something isn't working label Oct 3, 2023
@elsand elsand changed the title Fjern ETag fra samtlige request body i swagger Fjern ETag fra samtlige request body i swagger - generell swagger-cleanup Nov 10, 2023
@elsand
Copy link
Member

elsand commented Nov 10, 2023

Drar denne opp, vi trenger å prioritere å få en swagger som fungerer godt og etterhvert også har forklarende tekster og eksempler - målet er at denne skal kunne brukes til codegen og ende opp med noe som ser noenlunde fornuftig ut (se "Export" på https://app.swaggerhub.com/apis/Altinn/Dialogporten/v0.1).

Generelt er swaggeren en smule rotete, og har en hel del tomme og/eller ubrukte modeller med navn som SortablePaginationParameterOfListDialogQueryOrderDefinitionAndListDialogDto2, mens flere GET-operasjoner ikke har noe response-schema i det hele tatt. Virker som FastEndpoints.Swagger trenger å bli holdt mer i hånda.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants