metron-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (METRON-1895) Add Knox SSO as an option in Metron
Date Mon, 03 Dec 2018 13:51:01 GMT


ASF GitHub Bot commented on METRON-1895:

Github user justinleet commented on a diff in the pull request:
    --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/
    @@ -17,22 +17,66 @@
    +import org.springframework.beans.factory.annotation.Autowired;
    +import org.springframework.beans.factory.annotation.Value;
     import org.springframework.context.annotation.Bean;
     import org.springframework.context.annotation.Configuration;
    +import org.springframework.core.env.Environment;
     import org.springframework.web.bind.annotation.RestController;
     import springfox.documentation.spi.DocumentationType;
    +import springfox.documentation.spring.web.paths.RelativePathProvider;
     import springfox.documentation.spring.web.plugins.Docket;
     import springfox.documentation.swagger2.annotations.EnableSwagger2;
    +import javax.servlet.ServletContext;
    +import java.util.Arrays;
    +import java.util.List;
    +import static;
     public class SwaggerConfig {
    +  @Value("${knox.root}")
    --- End diff --
    In the interest of sparking a software religious war, apparently Spring recommends not
using field injection for Beans.
    I know we do it in multiple places, so I'm not necessarily concerned about fixing it here,
but I am interested in your thoughts on that since you're way more familiar with Spring than
I am.

> Add Knox SSO as an option in Metron
> -----------------------------------
>                 Key: METRON-1895
>                 URL:
>             Project: Metron
>          Issue Type: New Feature
>            Reporter: Ryan Merriman
>            Priority: Major
> This feature will enable accessing Metron REST and the UIs through Knox's SSO mechanism.

This message was sent by Atlassian JIRA

View raw message