Skip to content
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
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
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy