Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(144)

Unified Diff: net/base/dns_util_unittest.cc

Issue 201033: ForceTLS: hash hostnames, handle subdomains, canonicalise. (Closed)
Patch Set: ... Created 11 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/base/dns_util_unittest.cc
diff --git a/net/base/dns_util_unittest.cc b/net/base/dns_util_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..7995b922050a09dab51a706744a9e2ed138acf4f
--- /dev/null
+++ b/net/base/dns_util_unittest.cc
@@ -0,0 +1,60 @@
+// Copyright (c) 2009 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "net/base/dns_util.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace net {
+
+class DNSUtilTest : public testing::Test {
+};
+
+TEST_F(DNSUtilTest, DNSDomainFromDot) {
+ std::string out;
+
+ EXPECT_TRUE(DNSDomainFromDot("", &out));
+ EXPECT_EQ(out, "");
+ EXPECT_TRUE(DNSDomainFromDot("com", &out));
+ EXPECT_EQ(out, "\003com");
+ EXPECT_TRUE(DNSDomainFromDot("google.com", &out));
+ EXPECT_EQ(out, "\x006google\003com");
+ EXPECT_TRUE(DNSDomainFromDot("www.google.com", &out));
+ EXPECT_EQ(out, "\003www\006google\003com");
+
+ // Label is 63 chars: still valid
+ EXPECT_TRUE(DNSDomainFromDot("123456789a123456789a123456789a123456789a123456789a123456789a123", &out));
+ EXPECT_EQ(out, "\077123456789a123456789a123456789a123456789a123456789a123456789a123");
+
+ // Label is too long: invalid
+ EXPECT_FALSE(DNSDomainFromDot("123456789a123456789a123456789a123456789a123456789a123456789a1234", &out));
+
+ // 253 characters in the name: still valid
+ EXPECT_TRUE(DNSDomainFromDot("123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123", &out));
+ EXPECT_EQ(out, "\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\011123456789\003123");
+
+ // 254 characters in the name: invalid
+ EXPECT_FALSE(DNSDomainFromDot("123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.123456789.1234", &out));
+
+ // Zero length labels should be dropped.
+ EXPECT_TRUE(DNSDomainFromDot("www.google.com.", &out));
+ EXPECT_EQ(out, "\003www\006google\003com");
+
+ EXPECT_TRUE(DNSDomainFromDot(".google.com", &out));
+ EXPECT_EQ(out, "\006google\003com");
+}
+
+TEST_F(DNSUtilTest, STD3ASCII) {
+ EXPECT_TRUE(IsSTD3ASCIIValidCharacter('a'));
+ EXPECT_TRUE(IsSTD3ASCIIValidCharacter('b'));
+ EXPECT_TRUE(IsSTD3ASCIIValidCharacter('c'));
+ EXPECT_TRUE(IsSTD3ASCIIValidCharacter('1'));
+ EXPECT_TRUE(IsSTD3ASCIIValidCharacter('2'));
+ EXPECT_TRUE(IsSTD3ASCIIValidCharacter('3'));
+
+ EXPECT_FALSE(IsSTD3ASCIIValidCharacter('.'));
+ EXPECT_FALSE(IsSTD3ASCIIValidCharacter('/'));
+ EXPECT_FALSE(IsSTD3ASCIIValidCharacter('\x00'));
+}
+
+} // namespace net

Powered by Google App Engine
This is Rietveld 408576698