Skip to content

Fix resolve ContentType#206

Merged
runeflobakk merged 4 commits into
mainfrom
fix-ContentType-resolve
Oct 14, 2025
Merged

Fix resolve ContentType#206
runeflobakk merged 4 commits into
mainfrom
fix-ContentType-resolve

Conversation

@runeflobakk

Copy link
Copy Markdown
Member

Hovedsakelig fikser en bug (mulig den bare er latent, men vi traff den i en test som bruker biblioteket mot mocket endepunkt) af8c368 (forklaring i commit-melding)

Dro med seg litt mer opprydning i samme slengen.

The ContentType.create constructors are used for supplying the actual
constituents of a ContentType, and will fail if
ContentType.create(String) is passed e.g.
'application/xml; charset=UTF8'.

When reading a header value (i.e. parsing), ContentType.parse must be
used.
commons-logging is not longer a dependency of Apache HttpClient
@runeflobakk runeflobakk requested review from a team, eivinhb and shahzebs October 14, 2025 14:48

@ArneeMe ArneeMe left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

@runeflobakk runeflobakk merged commit 63044e0 into main Oct 14, 2025
4 checks passed
@runeflobakk runeflobakk deleted the fix-ContentType-resolve branch October 14, 2025 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants