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

Spoof validation #100

Merged
merged 3 commits into from May 9, 2016
Merged

Spoof validation #100

merged 3 commits into from May 9, 2016

Conversation

egulias
Copy link
Owner

@egulias egulias commented Apr 30, 2016

Closes #46
ping @masakielastic
I have removed the check for the ICU version since the restriction level has not been merged yet. When merged I'd add it.
Can you please confirm this code helps with spoof prevention?

@coveralls
Copy link

coveralls commented Apr 30, 2016

Coverage Status

Coverage decreased (-0.2%) to 91.98% when pulling f1116d3 on spoof_check_validation into 8c42133 on master.

@egulias egulias merged commit e4f04d5 into master May 9, 2016
@egulias egulias mentioned this pull request May 9, 2016
@egulias egulias deleted the spoof_check_validation branch May 9, 2016 22:00
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.

None yet

2 participants