Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was 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
Loading
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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.