-
Notifications
You must be signed in to change notification settings - Fork 0
Description
Description of the bug
Det er 2 konkrete bugs, men nært beslektet og løses nok samtidig:
Begge opptrer i ConfirmDownloaded, og fører til at FileTransfers slettes for tidlig / ikke når de skal, og at Grace period ikke respekteres, opprinnelig feature: #533
I praksis så går filer til Purge umiddelbart uten å håndheve PurgeFileTransferGracePeriod dersom PurgeFileTransferAfterAllRecipientsConfirmed = true.
PurgeFileTransferGracePeriod skal alltid håndheves, slik at ingen FileTransfers kan slettes FØR dette har utløpt
Den andre bug er mer alvorlig: dersom PurgeFileTransferAfterAllRecipientsConfirmed = false så skal man IKKE utføre Purge av FileTransfer før ExpiryDate (- noe som blir Enqued som en del av InitializeFileTransfer). - Dette skjer nå PurgeFileTransferGracePeriod etter AllConfirmedDownloaded ! - Så det oppfører seg som om PurgeFileTransferAfterAllRecipientsConfirmed = true.
Steps To Reproduce
Case 1:
- Initialize og upload FileTransfer på en Resource med PurgeFileTransferAfterAllRecipientsConfirmed = true og PurgeFileTransferGracePeriod = 24H og FileTransferTimeToLive= 48H
- Download og Confirm for alle Recipients
- Forventet: Purged først 24H etter AllConfirmDownloaded.
Faktisk: FileTransfer blir tilnærmet umiddelbart Purged
Case 2
- Initialize og upload FileTransfer på en Resource med PurgeFileTransferAfterAllRecipientsConfirmed = false, PurgeFileTransferGracePeriod = 24H og FileTransferTimeToLive= 48H
- Download og Confirm for alle Recipients
- Forventet: FileTransfer blir IKKE Purged før etter 48 timer.
Faktisk: FileTransfer blir Purged 24H etter AllConfirmDownloaded.
Additional Information
Mistenker at fiksen kan være å forenkle metoden her: https://github.com/Altinn/altinn-broker/blob/main/src/Altinn.Broker.Application/ConfirmDownload/ConfirmDownloadHandler.cs#L84
Metadata
Metadata
Assignees
Labels
Type
Projects
Status