Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

allows to test mongodb uri via env vars #174

Merged

Conversation

bizob2828
Copy link
Contributor

This restores the ability to pass in MONGOLAB_URI and MONGODB_URI as env vars to set the mongo db uri.

Fixes #173

@ckarande
Copy link
Member

@bizob2828 Thanks for the PR. LGTM

@UlisesGascon
Copy link
Collaborator

UlisesGascon commented Sep 30, 2019

Thanks @bizob2828 this PR seems good to me. I will move the target branch to OWASP:release-1.5. I will merge it tomorrow, so this PR will count you for Hacktoberfest 🎉

@UlisesGascon UlisesGascon changed the base branch from master to release-1.5 September 30, 2019 13:49
@UlisesGascon UlisesGascon added this to the v1.5 milestone Sep 30, 2019
@bizob2828
Copy link
Contributor Author

ok thanks @UlisesGascon. When will 1.5 be released? I'm currently just using the fork in our test harness but would love to restore using this main repo

@UlisesGascon UlisesGascon merged commit 2b7204d into OWASP:release-1.5 Oct 19, 2019
@UlisesGascon
Copy link
Collaborator

@bizob2828 not sure yet about the date. There are many issues and PRs on going. I will ping you back when we have an estimation 👍

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants