symfony · dsmink · Feb 9, 2018 · nicolas-grekas · Feb 9, 2018 · dsmink
diff --git a/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php b/src/Symfony/Component/DependencyInjection/EnvVarProcessor.php
@@ -111,11 +111,13 @@ public function getEnv($prefix, $name, \Closure $getEnv)
        }

        if ('const' === $prefix) {
+            $env = \str_replace("\\\\","\\", $env);
+
            if (!defined($env)) {
                throw new RuntimeException(sprintf('Env var "%s" maps to undefined constant "%s".', $name, $env));
            }

-            return constant($name);
+            return constant($env);
        }

        if ('base64' === $prefix) {
-Original file line number
+Diff line change
@@ -111,11 +111,13 @@ public function getEnv($prefix, $name, \Closure $getEnv)
             }
             if ('const' === $prefix) {
+                $env = \str_replace("\\\\","\\", $env);
             Copy link

  
      
    
  

  
      

  
  Member


      

  

  
    
      

      
            nicolas-grekas
  

      

      

      


        Feb 9, 2018


      
    

  


        
      
  
  
  
    

    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 should be removed (the blog post has a typo, the \ should not be doubled there, we're going to fix it)
    
  
  


    

        
      
  
  
    
  
  
  
    
    Sorry, something went wrong.
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.


  
  


          
      
  
    
    
      
        
            
    All reactions
  


          
          
        
      
    

    



  
        
  
    
        
    
  


      
          
  
      
            Copy link

  
      
    
  

  
      

  
    Contributor


      

  Author


  

  
    
      

      
            dsmink
  

      

      

      


        Feb 9, 2018


      
    

  


        
      
  
  
  
    

    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.
    

    
      
      


  


  
    
      We've tested here with and without the double \ but we could not get it to work.
The original scripts runs:                        return constant($name);

While I only could get it to work with:     return constante($env);
Will this be fixted to? Or are we doing something wrong?
    
  
  


    

        
      
  
  
    
  
  
  
    
    Sorry, something went wrong.
  

  
    
  
    
      

              Uh oh!

              
There was an error while loading. Please reload this page.


  
  


          
      
  
    
    
      
        
            
    All reactions
                 if (!defined($env)) {
                     throw new RuntimeException(sprintf('Env var "%s" maps to undefined constant "%s".', $name, $env));
                 }
-                return constant($name);
+                return constant($env);
             }
             if ('base64' === $prefix) {
-          Expand Down