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

Validator of MySQL's "host" field #521

Merged
merged 1 commit into from Feb 4, 2012

Conversation

Projects
None yet
2 participants
@neil-rubens
Copy link
Contributor

neil-rubens commented Feb 3, 2012

for the MySQL's "host" field
original validator (HOST_NAME_OR_IP_ADDRESS) does not seem to allow DNS hostnames;
e.g. address of MySQL hosted by Amazon's RDS service: db3.c32l4fl.ap-northeast-1.rds.amazonaws.com
gives an error: "Too many parts in IP address ..."

Fixed it by using the REQUIRE_NON_EMPTY_STRING validator instead.

for the MySQL's "host" field
original validator (HOST_NAME_OR_IP_ADDRESS) does not seem allow DNS hostnames; 
e.g. address of MySQL hosted by Amazon's RDS service: db3.c32l4fl.ap-northeast-1.rds.amazonaws.com 

Fixed it by using the REQUIRE_NON_EMPTY_STRING validator

@ghost ghost assigned mbastian Feb 4, 2012

mbastian added a commit that referenced this pull request Feb 4, 2012

Merge pull request #521 from neil-rubens/patch-1
Validator of MySQL's "host" field.

@mbastian mbastian merged commit c7a3440 into gephi:master Feb 4, 2012

@mbastian

This comment has been minimized.

Copy link
Member

mbastian commented Feb 4, 2012

Thanks Neil, that make sense to me.

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