Trimming strings while parsing the connection string to allow for more f... #65

Merged
merged 1 commit into from Sep 20, 2012

Conversation

Projects
None yet
2 participants
Contributor

dlbromen commented Sep 20, 2012

...lexibility with spaces around the connection string key/value pairs.

Accompanied with new unit tests.

This all stemmed from me accidentally putting something like the following in the connection string: "...Server=my-host-1, my-host-2, my-host-3;..." And because of those spaces DNS lookup was failing on all but the first in the list causing round-robin not to work.
I could see the same thing happening for other users on both the Servers tag and other key/value pairs so I think it makes sense to accommodate it.

@dlbromen dlbromen Trimming strings while parsing the connection string to allow for mor…
…e flexibility with spaces around the connection string key/value pairs.

Accompanied with new unit tests.
b42a36e

@nberardi nberardi added a commit that referenced this pull request Sep 20, 2012

@nberardi nberardi Merge pull request #65 from dlbromen/conn_str_parse
Trimming strings while parsing the connection string to allow for more f...
939d02e

@nberardi nberardi merged commit 939d02e into fluentcassandra:master Sep 20, 2012

Contributor

nberardi commented Sep 20, 2012

Thank you for the pull request

nberardi was assigned Oct 11, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment