Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[BUGFIX] Using a mysql socket specifier instead of a hostname didn't work properly #1

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

Conversation

kraftb
Copy link

@kraftb kraftb commented Aug 18, 2014

Mysql Credentials get used at 4 places in the extension. This patch unifies generating the mysql-tool credential string (--user=, --password=, etc.) and fixes the method to properly handle the case where typo_db_host is a socket specifier (i.e. :/var/run/mysqld/mysqld.sock)

@propheh
Copy link
Contributor

propheh commented Sep 5, 2014

Hi kraftb

I have an internal newer version (compatible with 6.2 and some other features), which I'll release as soon as I'm comfortable that it works. The patches should just be applicable 1:1, but please check once I released the newer version.

Cheers

@simonschaufi
Copy link

@propheh when is the new release released? quite some time has passed already...

@kraftb
Copy link
Author

kraftb commented Apr 9, 2015

We have an upgraded version of cabag_extbase and contentstage which should work even with 7.x. Let me know if you are interested. As this is work done for a customer of mine he maybe wants to charge a small fee for compensating the costs he had.

I know this is against the mind of open source. But sadly money becomes more and more a player in the OS world.

@kraftb
Copy link
Author

kraftb commented Apr 9, 2015

We shouldn't have let google in :)

@simonschaufi
Copy link

yes I'm interested. You can tell your customer, that I will also contribute back and will add more features because coparing 2 databases is horribly slow at the moment. I would rather do everything locally instead. I'm already working on that...

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.

3 participants