Github user rmetzger commented on a diff in the pull request:

    https://github.com/apache/flink/pull/2425#discussion_r77320094
  
    --- Diff: 
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/HttpRequestHandler.java
 ---
    @@ -99,7 +110,43 @@ public void channelRead0(ChannelHandlerContext ctx, 
HttpObject msg) {
                                        currentDecoder.destroy();
                                        currentDecoder = null;
                                }
    -                           
    +
    +                           if(secureCookie != null) {
    --- End diff --
    
    The YARN code does the auto-populaton only if security is enabled, so it 
should not be an issue.
    
    > user-provided but will be persisted in the in-memory Flink configuration 
instance which is passed to the web layer during bootstrap.
    
    The Configuration object in the CliFrontend is not transferred to the 
ApplicationMaster / JobMaster.
    The flink-config.yaml (and on YARN the "dynamic properties") are the only 
sources of the configuration for the AM/JM.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to