-
Notifications
You must be signed in to change notification settings - Fork 76
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
Remake console backend from JAX-RS to Spring-MVC #6755
Remake console backend from JAX-RS to Spring-MVC #6755
Conversation
…or spring endpoints & remade SecurityItems endpoint in spring
FileViewer in spring does currently stream
…g-web-mvc' of https://github.com/frankframework/frankframework into issue/6676-migrate-console-backend-from-jax-rs-to-spring-web-mvc
…x-rs-to-spring-web-mvc
…nd-from-jax-rs-to-spring-web-mvc' into issue/6676-migrate-console-backend-from-jax-rs-to-spring-web-mvc
…g-web-mvc' of https://github.com/frankframework/frankframework into issue/6676-migrate-console-backend-from-jax-rs-to-spring-web-mvc
SEC_LOG.info("created bus request from URI [{}:{}] issued by{} with headers [{}] payload [{}]", method, base.getServletRequest() | ||
.getRequestURI(), issuedBy, headers, payload); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks
if("GET".equalsIgnoreCase(method) || "OPTIONS".equalsIgnoreCase(method)) { | ||
SEC_LOG.debug("created bus request from URI [{}:{}] issued by{}", method, base.getUriInfo().getRequestUri(), issuedBy); | ||
if ("GET".equalsIgnoreCase(method) || "OPTIONS".equalsIgnoreCase(method)) { | ||
SEC_LOG.debug("created bus request from URI [{}:{}] issued by{}", method, base.getServletRequest().getRequestURI(), issuedBy); |
Check notice
Code scanning / SonarCloud
Logging should not be vulnerable to injection attacks
// This event listens to all Spring refresh events. | ||
// When adding new Spring contexts (with this as a parent) refresh events originating from other contexts will also trigger this method. | ||
// Since we never want to reinitialize this servlet, we can ignore the 'refresh' event completely! | ||
super.doService(request, response); |
Check notice
Code scanning / SonarCloud
Exceptions should not be thrown from servlet methods
…nd-from-jax-rs-to-spring-web-mvc' into issue/6676-migrate-console-backend-from-jax-rs-to-spring-web-mvc
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.
Ik heb niet alle tests bekeken maar voor de rest lijkt het allemaal naar behoren te werken 🎉
console/backend/src/main/java/org/frankframework/management/web/ConfigurationStatus.java
Show resolved
Hide resolved
console/backend/src/main/java/org/frankframework/management/web/FrankApiBase.java
Outdated
Show resolved
Hide resolved
console/backend/src/main/java/org/frankframework/management/web/Init.java
Show resolved
Hide resolved
console/backend/src/main/java/org/frankframework/management/web/Monitors.java
Show resolved
Hide resolved
...nd/src/main/java/org/frankframework/management/web/configuration/DeprecationInterceptor.java
Show resolved
Hide resolved
|
||
import org.springframework.web.filter.ShallowEtagHeaderFilter; | ||
|
||
public class WeakShallowEtagHeaderFilter extends ShallowEtagHeaderFilter { |
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.
Adding this reason in this PR is nice, but won't be seen in the future. Please use Javadoc instead.
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
No description provided.