Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Migration to Micronaut 4 #357

Merged
merged 7 commits into from
Nov 10, 2022
Merged

Migration to Micronaut 4 #357

merged 7 commits into from
Nov 10, 2022

Conversation

dstepanov
Copy link
Contributor

@dstepanov dstepanov commented Nov 7, 2022

PR uses refactored routing API from Core eliminating duplicate code.

There are some unresolved errors caused by CORS not doing the same as before and errors caused by serde migrations.

@dstepanov
Copy link
Contributor Author

@sdelamo Can you please investigate the CORS errors?

@sdelamo
Copy link
Contributor

sdelamo commented Nov 7, 2022

@sdelamo Can you please investigate the CORS errors?

@timyates could you do that?

@timyates
Copy link
Contributor

timyates commented Nov 7, 2022

Ok, so I've take a look at the CORS failures...

What I suspect is happening is that HttpClient has changed to keep-alive as default

micronaut-projects/micronaut-core#8100 (comment)

And Jetty doesn't return Connection: keep-alive headers, as that is the default for HTTP 1.1

I'll go with this, and get a commit together that goes with this expectation

Now we have a keep-alive client, the Connection is no longer returned by Jetty
@timyates
Copy link
Contributor

timyates commented Nov 7, 2022

@dstepanov Fixed the cors errors...

There are still binding errors, and fields not being bound from Json

Can you take a look?

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 8, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 19 Code Smells

71.0% 71.0% Coverage
0.0% 0.0% Duplication

@graemerocher graemerocher merged commit f24f3a8 into master Nov 10, 2022
@graemerocher graemerocher deleted the migration4 branch November 10, 2022 14:28
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.

4 participants