New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 722174 link

Starred by 1 user
Status: Started
Owner:
█ mgiuca@chromium.org
Long OOO (go/where-is-mgiuca)
Cc:
js...@chromium.org
pkasting@chromium.org
█ mgiuca@chromium.org
creis@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Feature
Team-Security-UX



Sign in to add a comment

Added a Python script to generate IDN test cases

Project Member Reported by mgiuca@chromium.org, May 15 2017 
Currently, the IDN test cases:
components/url_formatter/url_formatter_unittest.cc

Are generated by hand. (Or maybe jshin@ has a script for it.)

It would be helpful if we had a tool to automate this; in particular, making sure the first element of the IDNTestCase object is a valid IDN encoding of a Unicode domain, and that the second element is normalized according to IDNA (NFD I believe).
Project Member

Comment 1 by bugdroid1@chromium.org, May 18 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ab5dea2383c21e0fe560357cd9468b19af628510

commit ab5dea2383c21e0fe560357cd9468b19af628510
Author: mgiuca <mgiuca@chromium.org>
Date: Thu May 18 06:34:35 2017

Added a Python script in tools/ to generate IDN test cases.

This script takes a Unicode domain as input, and generates the test case
in C++ syntax, automatically generating the IDN encoding and
normalization of the domain, and converting to a C++ wide string
literal.

BUG=722174

Review-Url: https://codereview.chromium.org/2880983002
Cr-Commit-Position: refs/heads/master@{#472709}

[modify] https://crrev.com/ab5dea2383c21e0fe560357cd9468b19af628510/components/url_formatter/url_formatter_unittest.cc
[add] https://crrev.com/ab5dea2383c21e0fe560357cd9468b19af628510/tools/security/idn_test_case_generator.py

Comment 2 by est...@chromium.org, Nov 10 2017

Labels: Hotlist-EnamelAndFriendsFixIt

Comment 3 by est...@chromium.org, Feb 18 2018

Labels: -Hotlist-EnamelAndFriendsFixIt
►

Sign in to add a comment