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

Ip validation #285

Closed
pradeepkumars opened this issue Dec 17, 2013 · 2 comments
Closed

Ip validation #285

pradeepkumars opened this issue Dec 17, 2013 · 2 comments

Comments

@pradeepkumars
Copy link
Contributor

IP validation fails if uder provide below IP for template, nfs pool.

101.10.10.10

It accepts 3 digits also

Ideal check would be:

!/^((2[0-4]\d|25[0-5]|[01]?\d\d?).){3}(2[0-4]\d|25[0-5]|[01]?\d\d?

If java script provide any other better check for ipv4/ipv6, that would be ever more better.

@ghost ghost assigned zhoumeina Jan 22, 2014
@shaohef shaohef assigned YuXinHuo and unassigned zhoumeina Feb 24, 2014
@shaohef
Copy link

shaohef commented Feb 24, 2014

@YuXinHuo I know you provides IP validation for network.
Then how about add it for template and nfs pool.

@alinefm alinefm added the bug label Mar 3, 2014
@alinefm alinefm added this to the 1.2.1 GA milestone Apr 1, 2014
@adamkingit adamkingit assigned hlwanghl and unassigned YuXinHuo Apr 2, 2014
@alinefm alinefm removed this from the 1.2.1 GA milestone Jul 2, 2014
@alinefm alinefm removed the storage label Sep 24, 2014
@alinefm alinefm added this to the 1.4 GA milestone Dec 3, 2014
@alinefm alinefm assigned ramonmedeiros and unassigned hlwanghl Dec 3, 2014
@alinefm alinefm modified the milestones: Backlog, 1.4 GA Dec 10, 2014
@alinefm
Copy link
Member

alinefm commented Dec 18, 2015

It is part of #37

@alinefm alinefm closed this as completed Dec 18, 2015
@alinefm alinefm added duplicated and removed bug labels Dec 18, 2015
@alinefm alinefm removed this from the Backlog milestone Dec 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants