Skip to content

Closing external acces to DB#11

Merged
Yenthe666 merged 1 commit intoYenthe666:9.0from
sebastian-code:patch-3
Oct 12, 2015
Merged

Closing external acces to DB#11
Yenthe666 merged 1 commit intoYenthe666:9.0from
sebastian-code:patch-3

Conversation

@sebastian-code
Copy link
Contributor

With those lines you were able to access the database engine externally, something you want not in a production environment.

With those lines you were able to access the database engine externally, something you want not in a production environment.
@Yenthe666
Copy link
Owner

@sebastian-code this could be a problem indeed. Never noticed this myself!
I'll give this a little testrun tommorow and if everything is fine I'll merge it in.

@Yenthe666
Copy link
Owner

@sebastian-code good catch and absolutely true. I see no problems in taking this out. Thanks! 👍

Yenthe666 added a commit that referenced this pull request Oct 12, 2015
@Yenthe666 Yenthe666 merged commit b0faff0 into Yenthe666:9.0 Oct 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants