Opened 13 years ago

Last modified 13 years ago

#38 closed defect (FIXED)

Cannot specify custom monod port

Reported by: Sebastian Spaeth Owned by: Christopher Allan Webber
Priority: minor Milestone:
Component: Keywords:
Cc: Parent Tickets:

Description

I add

[app:mediagoblin]
...
db\_port = 27017

to use a custom mongod port (it wasn't finding the running one and
I don't know which is the default it tries).

I get

::

      File "/home/spaetz/src/mediagoblin/eggs/PasteDeploy-1.3.4-py2.7.egg/paste/deploy/util/fixtypeerror.py", line 57, in fix_call
        val = callable(*args, **kw)
      File "/home/spaetz/src/mediagoblin/mediagoblin/app.py", line 118, in paste_app_factory
        app_config.get('db_host'), app_config.get('db_port'))
      File "/home/spaetz/src/mediagoblin/eggs/mongokit-0.6-py2.7.egg/mongokit/connection.py", line 52, in __init__
        super(Connection, self).__init__(*args, **kwargs)
      File "/usr/lib/pymodules/python2.7/pymongo/connection.py", line 242, in __init__
        raise TypeError("port must be an instance of int")
    TypeError: port must be an instance of int

I guess there is a missing int() wrapper somewhere.



Change History (3)

comment:1 by Sebastian Spaeth, 13 years ago

patch offered here for merging:
`https://gitorious.org/mediagoblin/mediagoblin/merge\_requests/2 <https://gitorious.org/mediagoblin/mediagoblin/merge_requests/2>`_



comment:2 by Christopher Allan Webber, 13 years ago

Status: NewClosed
Thanks, fixed, merged, pushed. I also added it to
mediagoblin.celery\_setup.from\_celery, made it not fail if the
port wasn't specified, and used the paste converter "asint" which I
think gives a nicer error when things don't work.

Thanks for submitting this bug report and for the fix. :)



comment:3 by Will Kahn-Greene, 12 years ago

The original url for this bug was http://bugs.foocorp.net/issues/309 .

Note: See TracTickets for help on using tickets.