[bbDev] installation issue

[Sam Bauers]

Short regexp primer... plus potential bug...

The regexp wants there to be at least one period (.) in there.

(.*\.)?([^.]+\.[^.]+)

Break it up into parts

(           <-- start subpattern
    .*       <-- 0 or more of any character
    \.       <-- exactly one period
)?          <-- end subpattern (match 0 or 1 of this subpattern)

(           <-- start subpattern
    [^.]+    <-- 1 or more of any character that is not a period
    \.       <-- exactly one period
    [^.]+    <-- 1 or more of any character that is not a period
)           <-- end subpattern (match 1 of this subpattern)

"localhost" does not satisfy this pattern, "four.three.two.one" does,  
as does "two.one"

Looking at that code, the next bit of logic compares the bbpress  
domain to the wordpress domain to test for cookie sharing, but it  
appears to only test the last two parts of the domain names. IE if  
your bbpress domain name is www.example.org and your wordpress domain  
name is www.example.org it tests them by comparing whether  
"example.org" matches "example.org".

This seems a bit dodgy as down here where I am and in other parts of  
the word we have domain names like www.example.org.au, so if we do a  
similar test we are checking "org.au" matches "org.au". So the test  
will potentially not return an error when it should, like if your  
bbpress domain name is www.example.org.au and your wordpress domain  
name is www.another.org.au, by the looks of things it will match  
"org.au" to "org.au" and keep you blissfully unaware that the  
integration won't work.

I can't suggest a better test because I'm not sure of the  
implications it may have to later setting of the cookies domain.

Sam




On 08/02/2007, at 1:21 PM, Brad Bolman wrote:

> Ah...that changes things.
>
> On 2/7/07, Aditya Naik <so1oonnet at gmail.com> wrote: well this is a  
> local installation on my machine..
>
>
> On 2/7/07, Brad Bolman < bbolman at gmail.com> wrote: for your $bb- 
> >domain: try http://www.yoursite.com
>
> and for the WP settings, http://www.yoursite.com/wordpress
>
> If that's already what you have pardon me, but usually folks don't  
> use localhost (or at least in my experience.
>
> On 2/7/07, Aditya Naik < so1oonnet at gmail.com> wrote:
> ok.. at the risk of making a complete idiot of myself - i proclaim  
> - i have an issue
>
> here is the related config file options:
>     // Just the domain name; no directories or path. There should  
> be no trailing slash here.
>     $bb->domain = 'http://localhost'; // Example: ' http:// 
> bbpress.example.com'
>     // There should be both a leading and trailing slash here. '/'  
> is fine if the site is in root.
>     $bb->path   = '/wordpress/forums/';                   //  
> Example: '/forums/'
>
> $bb->wp_home = ' http://localhost/wordpress';  // WordPress -  
> Options->General: Blog address (URL) // No trailing slash
> $bb->wp_siteurl = 'http://localhost/wordpress';  // WordPress -  
> Options->General: WordPress address (URL) // No trailing slash
>
> and now when i try to install it gives me this
> First Step
> Make sure you have everything (database information, email address,  
> etc.) entered correctly in config.php before running this script.
>
> Your $bb->domain setting cannot be parsed.
> Current settings
> $bb->domain:http://localhost
>
> and with my limited knowledge of reg exp i am stumped on line 109!
>
> any one?
>
>
>
> -- 
> drop by at http://www.adityanaik.com
> _______________________________________________
> bbDev mailing list
> bbDev at lists.bbpress.org
> http://lists.bbpress.org/mailman/listinfo/bbdev
>
>
>
> _______________________________________________
> bbDev mailing list
> bbDev at lists.bbpress.org
> http://lists.bbpress.org/mailman/listinfo/bbdev
>
>
>
>
> -- 
> drop by at http://www.adityanaik.com
>
> _______________________________________________
> bbDev mailing list
> bbDev at lists.bbpress.org
> http://lists.bbpress.org/mailman/listinfo/bbdev
>
>
> _______________________________________________
> bbDev mailing list
> bbDev at lists.bbpress.org
> http://lists.bbpress.org/mailman/listinfo/bbdev