You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by "impetus technologies (JIRA)" <ji...@apache.org> on 2008/06/26 13:21:46 UTC

[jira] Created: (SHINDIG-409) base_path in container.php is not correct

base_path in container.php is not correct
-----------------------------------------

                 Key: SHINDIG-409
                 URL: https://issues.apache.org/jira/browse/SHINDIG-409
             Project: Shindig
          Issue Type: Bug
         Environment: Windows, Apache
            Reporter: impetus technologies
         Attachments: container.php.patch

The base directory is PHP then base_path should be upto /php not upto /php/config

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (SHINDIG-409) base_path in container.php is not correct

Posted by "impetus technologies (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-409?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

impetus technologies updated SHINDIG-409:
-----------------------------------------

    Attachment: container.php.patch

> base_path in container.php is not correct
> -----------------------------------------
>
>                 Key: SHINDIG-409
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-409
>             Project: Shindig
>          Issue Type: Bug
>         Environment: Windows, Apache
>            Reporter: impetus technologies
>         Attachments: container.php.patch
>
>
> The base directory is PHP then base_path should be upto /php not upto /php/config

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Resolved: (SHINDIG-409) base_path in container.php is not correct

Posted by "Chris Chabot (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/SHINDIG-409?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Chris Chabot resolved SHINDIG-409.
----------------------------------

    Resolution: Fixed
      Assignee: Chris Chabot

Woops guess i missed that one accidently, nice catch!

While fixing that i decided to clean up the other path references too, realpath should really be around the whole path statement so it resolves all the /../ bits for us as well and leaves us with clean /root/to/shindig/javascipt kind of path's without ..'s

So the patch became:
-       'base_path' => realpath(dirname(__FILE__)),
-       'features_path' => realpath(dirname(__FILE__)) . '/../../features/',
-       'container_path' => realpath(dirname(__FILE__)) . '/../../config/',
-       'javascript_path' => realpath(dirname(__FILE__)) . '/../../javascript/',
-       'container_config' => realpath(dirname(__FILE__)) . '/../../config/container.js',
+       'base_path' => realpath(dirname(__FILE__).'/..').'/',
+       'features_path' => realpath(dirname(__FILE__) . '/../../features').'/',
+       'container_path' => realpath(dirname(__FILE__) . '/../../config').'/',
+       'javascript_path' => realpath(dirname(__FILE__) . '/../../javascript').'/',
+       'container_config' => realpath(dirname(__FILE__) . '/../../config').'/container.js',
 
        // The OAuth SSL certificates to use, and the pass phrase for the private key  
-       'private_key_file' => realpath(dirname(__FILE__)) . '/../certs/private.key', 
-       'public_key_file' => realpath(dirname(__FILE__)) . '/../certs/public.crt', 
+       'private_key_file' => realpath(dirname(__FILE__) . '/../certs').'/private.key', 
+       'public_key_file' => realpath(dirname(__FILE__) . '/../certs').'/public.crt', 
        'private_key_phrase' => 'partuza', 

Thanks again for the great bug tracking!

> base_path in container.php is not correct
> -----------------------------------------
>
>                 Key: SHINDIG-409
>                 URL: https://issues.apache.org/jira/browse/SHINDIG-409
>             Project: Shindig
>          Issue Type: Bug
>         Environment: Windows, Apache
>            Reporter: impetus technologies
>            Assignee: Chris Chabot
>         Attachments: container.php.patch
>
>
> The base directory is PHP then base_path should be upto /php not upto /php/config

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.