Juster opplasting for sikre at API-lesere aldri ser dokumentobjekt uten fil - #317
Open
petterreinholdtsen wants to merge 1 commit into
Open
Conversation
…en fil. Noark 5 krever at det er en fil koblet til hver dokumentobjekt-instans, men det vil være en periode mellom dokumentobjekt blir opprettet og en fil blir lastet opp der et slikt dokumentobjekt mangler slik kobling. For å sikre at ingen API-leser ser slike inkonsistente dokumentobjekt-instanser, gjør det klart at slike ikke skal returneres som barn av sin foreldre-dokumentbeskrivelse før filopplastingen har lykkes. Relatert til mangelmelding nasjonalarkivet#285 og nasjonalarkivet#25, og tilbyr en bakoverkompatibel løsning uten ekstrafunksjonaliteten beskrevet i nasjonalarkivet#309 og nasjonalarkivet#298.
petterreinholdtsen
changed the title
Collaborator
Author
|
Jeg møtte denne situasjonen i Nikita under testing de siste dagene, der revers-mellomtjeneren nginx, nektet opplasting av filer større enn 1 MiB (filen var 1.5 MiB), noe som førte til at import-email døde med 413 Request Entity Too Large, og export-all --xml døde da den traff på en dokumentobjekt-instans uten sjekksum. Et spørsmål er om slikt skal måtte håndteres av API-klienter eller API-tjeneste. |
Contributor
|
Denne er grei. Trenger litt videre arbeid i nikita for å sørge for at nikita overholder det riktig. |
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
2 participants
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.
Noark 5 krever at det er en fil koblet til hver dokumentobjekt-instans, men det vil være en periode mellom dokumentobjekt blir opprettet og en fil blir lastet opp der et slikt dokumentobjekt mangler slik kobling. For å sikre at ingen API-leser ser slike inkonsistente dokumentobjekt-instanser, gjør det klart at slike ikke skal returneres som barn av sin foreldre-dokumentbeskrivelse før filopplastingen har lykkes.
Relatert til mangelmelding #285 og #25, og tilbyr en bakoverkompatibel løsning uten ekstrafunksjonaliteten beskrevet i #309 og #298.