-
Notifications
You must be signed in to change notification settings - Fork 19
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
Upgrade to jakarta.servlet namespace #402
Conversation
❌ Java CI failed: https://ge.micronaut.io/s/oehz5fadvrw44 |
❌ GraalVM CE CI 17 latest failed: https://ge.micronaut.io/s/w5it655amdgre |
transformation = MapFormat.MapTransformation.FLAT, | ||
keyFormat = StringConvention.RAW) Map<String, String> attributes) { | ||
if (CollectionUtils.isNotEmpty(attributes)) { | ||
attributes.forEach(tomcatConnector::setAttribute); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this no longer exists on Tomcat
try { | ||
if (LOG.isDebugEnabled()) { | ||
LOG.debug("Request [{} - {}] completed successfully", req.getMethodName(), req.getUri()); | ||
LOG.debug("Request [{} - {}] completed successfully", methodName, uri); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tomcat now errors if the request is accessed after completion, this fixes that
@@ -79,7 +81,7 @@ | |||
* @since 1.0.0 | |||
*/ | |||
@Internal | |||
public class DefaultServletHttpRequest<B> implements | |||
public class DefaultServletHttpRequest<B> extends MutableConvertibleValuesMap<Object> implements |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could no longer user servlet request attributes because they can't be accessed after the request completes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why SonarQube failed 🤔
Upgrade to the
jakarta.servlet
namespace and upgrade all Servlet modules