[Webkit-unassigned] [Bug 26154] SecurityOrigin::createFromDatabaseIdentifier should handle _'s in the hostname

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 3 09:05:10 PDT 2009


darin at apple.com changed:

           What    |Removed                     |Added
  Attachment #30891|review?                     |review+
               Flag|                            |

------- Comment #2 from darin at apple.com  2009-06-03 09:05 PDT -------
(From update of attachment 30891)

This is something you're supposed to delete if you either add a test case or
decide you can't add one. You shouldn't submit patches with this in them. We
don't want to check this warning into the change log!

> +    // Ensure there were at least 2 seperator characters.  Some hostnames on intranets have
> +    // underscores in them, so we'll assume that any additional underscores are part of the host.

Typo in this comment ("seperator"). Also, we use only one space after periods
in our comments.

Seems OK to land this without a test. To make a test we'd have to change
run-webkit-tests to set up the Apache server to run with a hostname that had an
underscore in it. Not impossible, but perhaps difficult.

It would also be good to have tests for the existing behavior, for example,
illegal database identifiers that fall into the failure case.


Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

More information about the webkit-unassigned mailing list