Skip to content
This repository has been archived by the owner on Aug 26, 2021. It is now read-only.

Fix for issue #33, parse MongoDB URL with multiple nodes #34

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kierandarcy
Copy link

No description provided.

@singingwolfboy
Copy link
Contributor

This looks rather complicated. I'm not familiar with mongodb connection URLs. Can you give me a concrete example of a connection URL that Flask-Heroku is currently failing to parse?

@kierandarcy
Copy link
Author

Thank you for taking the time to look at this. This is an example of an actual URL which Flask-Heroku is currently failing to parse.

mongodb://username:password@ds012345-a0.mlab.com:12345,ds012345-a1.mlab.com:12345/database?replicaSet=rs-ds012345

(Python's builtin urlparse will fail when trying to parse the port in this URL.)

@kierandarcy kierandarcy reopened this Mar 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants