Skip to content

Fix bug where FileItem returns empty bytearray #130

Merged
thburnett merged 12 commits into
mainfrom
dev/test-signature-bug
Jun 6, 2025
Merged

Fix bug where FileItem returns empty bytearray #130
thburnett merged 12 commits into
mainfrom
dev/test-signature-bug

Conversation

@thburnett
Copy link
Copy Markdown
Contributor

@thburnett thburnett commented Apr 9, 2025 •
edited
Loading

https://github.com/navikt/team-emottak-docs/issues/181

Etter oppgraderingen av ktor fra 2 til 3, har payloads sendt som MultiPart FileItem blitt tom array, som gjør at videre prosessering feiler.

@thburnett thburnett marked this pull request as ready for review April 11, 2025 08:16
@thburnett thburnett requested a review from a team as a code owner April 11, 2025 08:16
ivanskodje
ivanskodje approved these changes Apr 24, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@ivanskodje ivanskodje left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jeg har sett på dette kanskje 2 eller 3 separate ganger, fordi det er vanskelig å se om dette blir rett/løse problemet bare fra koden uten å teste dette manuelt. Siden ingenting så veldig galt ut, sier jeg bare LGTM! 🔥

@thburnett
Copy link
Copy Markdown
Contributor Author

thburnett commented Apr 24, 2025

Jeg har sett på dette kanskje 2 eller 3 separate ganger, fordi det er vanskelig å se om dette blir rett/løse problemet bare fra koden uten å teste dette manuelt. Siden ingenting så veldig galt ut, sier jeg bare LGTM! 🔥

Det skjønner jeg godt. Det er lagt til tester som skal sørge for at meldingene leses inn korrekt, men det er ikke helt intuitivt hva som skjer der. Kan ta en gjennomgang av problemstillingen og endringen på delingsmøtet neste uke.

@OleksandrChmyrNAV
Copy link
Copy Markdown
Contributor

OleksandrChmyrNAV commented May 2, 2025

Bare en liken headsup.
Endringer er i ebms-provide, så man må deaktivere hendelseshåndtering før prodsetting av denne fiksen.
Eller prodsette infrastruktur (topicer) for hendelseshåndtering på forhånd.

thburnett added 3 commits May 28, 2025 13:34
@thburnett
Merge branch 'main' into dev/test-signature-bug
c806035 
# Conflicts:
#	ebms-provider/src/main/kotlin/no/nav/emottak/ebms/EbmsDocumentExchange.kt
@thburnett
Merge branch 'main' into dev/test-signature-bug
298f9a9
@thburnett
Fix test
0907b22
@RettIProd RettIProd self-requested a review June 2, 2025 14:33
@thburnett thburnett merged commit 7cd2941 into main Jun 6, 2025
8 of 13 checks passed
@terjenilssen terjenilssen deleted the dev/test-signature-bug branch July 23, 2025 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RettIProd RettIProd Awaiting requested review from RettIProd

1 more reviewer

@ivanskodje ivanskodje ivanskodje approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@thburnett @OleksandrChmyrNAV @ivanskodje