Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Flyttet jetty over til 12. Obs på endring i jetty pakker. #1213

Merged
merged 3 commits into from
Sep 10, 2023
Merged

Conversation

mrsladek
Copy link
Contributor

@mrsladek mrsladek commented Sep 10, 2023

Testet lokalt.

@JsonCreator krever at ekstra annoteringer settes på parameterne siden man ikke kan aksessere parameter navn over reflection ved runtime.

Jetty 12 krever en import av spesiele EE10 pakker fra

  <!-- Jetty BOM -->
  <dependency>
      <groupId>org.eclipse.jetty.ee10</groupId>
      <artifactId>jetty-ee10-bom</artifactId>
      <version>${jetty.version}</version>
      <scope>import</scope>
      <type>pom</type>
  </dependency>

Det ser ut at jetty 12 skal støtte både EE8, EE9 og EE10.

@mrsladek mrsladek marked this pull request as ready for review September 10, 2023 16:35
@mrsladek mrsladek requested review from a team as code owners September 10, 2023 16:35
@jolarsen
Copy link
Contributor

Klarer vi oss uten JsonCreator her? Har litt å si for resten av apps.
Skal vi la EE10 og Jetty12 ligge til oktober som tidligere diskutert - først gjøre TFP + sanere WS. Deretter fokus på oppgradering

Copy link
Contributor

@jolarsen jolarsen left a comment

Choose a reason for hiding this comment

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

Se kommentar i tråd

@mrsladek
Copy link
Contributor Author

@jolarsen ja VTP er bare en test polygon for meg til å samle erfaring. Resten tar vi etter WSene er borte.

@mrsladek
Copy link
Contributor Author

Jeg er forresten noe usikker hvorfor @JsonCreator fungerte uten de ekstra annoteringer med EE9 men ikke med EE10.

@jolarsen
Copy link
Contributor

Normalt sett er det Jackson som oppretter ting. Jeg lurer på om Jetty eller Jersey bruker andre json-mappers og det kan være versjonsforskjeller.
Bruker vi Jersey 3.1.3 i VTP?

@jolarsen
Copy link
Contributor

@mrsladek Er det ikke en ganske gammel fp-bom i vtp? 1.0.0 .... Siste fp-bom har jersey 3.0.11 mens EE10 liker Jersey 3.1.3

Comment on lines -7 to -12
<parent>
<groupId>no.nav.foreldrepenger.felles</groupId>
<artifactId>fp-bom</artifactId>
<version>1.0.0</version>
</parent>

Copy link
Contributor Author

@mrsladek mrsladek Sep 10, 2023

Choose a reason for hiding this comment

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

@jolarsen Jeg har fjernet bruk av fp-bom her.

Copy link
Contributor

@espenwaaga espenwaaga left a comment

Choose a reason for hiding this comment

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

LGTM

@mrsladek mrsladek merged commit 095a569 into master Sep 10, 2023
@mrsladek mrsladek deleted the ee10 branch September 10, 2023 19:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants