You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@couchdb.apache.org by "Dean Landolt (JIRA)" <ji...@apache.org> on 2009/01/17 20:38:59 UTC

[jira] Created: (COUCHDB-213) config test failure for ports 80 and 443

config test failure for ports 80 and 443
----------------------------------------

                 Key: COUCHDB-213
                 URL: https://issues.apache.org/jira/browse/COUCHDB-213
             Project: CouchDB
          Issue Type: Bug
          Components: Test Suite
            Reporter: Dean Landolt
            Priority: Minor


In the config test this assertion:

T(config.httpd.port == port);

depends on the line:

var port = CouchDB.host.split(":").pop();

 But there's no :port on when running on 80 and 443. If config.httpd.port == 80 or 443 the port assertion should be skipped (and perhaps the protocol path part checked for http:// or https://)

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

[jira] Commented: (COUCHDB-213) config test failure for ports 80 and 443

Posted by "Glenn Rempe (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/COUCHDB-213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12667049#action_12667049 ] 

Glenn Rempe commented on COUCHDB-213:
-------------------------------------

I have updated my couchdb, which sits behind a proxy, and I am still seeing the same behavior.

http://couchdb.rempe.org/_utils/couch_tests.html?script/couch_tests.js

This instance was built from source with commit ID trunk@737346

I can give you admin access for this instance.  Please contact me by email and I'll give you a password if you'd like to use it for testing.

I am still seeing the error message:

# Assertion failed: config.httpd.port == port

And I have confirmed that the new test is the one that is running.  I ran this in FireFox (latest) on Mac OS X.  I cleared cache before running.

Perhaps this line is not evaluating correctly?  In which case I think the behavior would be identical to the old (broken) behavior.

if(server_port.length == 1 && CouchDB.inBrowser) {

Please re-open.

Thanks.

> config test failure for ports 80 and 443
> ----------------------------------------
>
>                 Key: COUCHDB-213
>                 URL: https://issues.apache.org/jira/browse/COUCHDB-213
>             Project: CouchDB
>          Issue Type: Bug
>          Components: Test Suite
>            Reporter: Dean Landolt
>            Priority: Minor
>
> In the config test this assertion:
> T(config.httpd.port == port);
> depends on the line:
> var port = CouchDB.host.split(":").pop();
>  But there's no :port on when running on 80 and 443. If config.httpd.port == 80 or 443 the port assertion should be skipped (and perhaps the protocol path part checked for http:// or https://)

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

[jira] Commented: (COUCHDB-213) config test failure for ports 80 and 443

Posted by "Jan Lehnardt (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/COUCHDB-213?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12674289#action_12674289 ] 

Jan Lehnardt commented on COUCHDB-213:
--------------------------------------

We could just drop the test for the port number as we are testing other config items as well that should pass. We probably should have done this in the first place. Any objections?

> config test failure for ports 80 and 443
> ----------------------------------------
>
>                 Key: COUCHDB-213
>                 URL: https://issues.apache.org/jira/browse/COUCHDB-213
>             Project: CouchDB
>          Issue Type: Bug
>          Components: Test Suite
>            Reporter: Dean Landolt
>            Priority: Minor
>
> In the config test this assertion:
> T(config.httpd.port == port);
> depends on the line:
> var port = CouchDB.host.split(":").pop();
>  But there's no :port on when running on 80 and 443. If config.httpd.port == 80 or 443 the port assertion should be skipped (and perhaps the protocol path part checked for http:// or https://)

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

[jira] Closed: (COUCHDB-213) config test failure for ports 80 and 443

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

Jan Lehnardt closed COUCHDB-213.
--------------------------------

    Resolution: Fixed

Closed as of r737346

> config test failure for ports 80 and 443
> ----------------------------------------
>
>                 Key: COUCHDB-213
>                 URL: https://issues.apache.org/jira/browse/COUCHDB-213
>             Project: CouchDB
>          Issue Type: Bug
>          Components: Test Suite
>            Reporter: Dean Landolt
>            Priority: Minor
>
> In the config test this assertion:
> T(config.httpd.port == port);
> depends on the line:
> var port = CouchDB.host.split(":").pop();
>  But there's no :port on when running on 80 and 443. If config.httpd.port == 80 or 443 the port assertion should be skipped (and perhaps the protocol path part checked for http:// or https://)

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