Increase max local-part length to 128

- Improves compatibility with Mailgun
- Closes #41
- Update CHANGELOG

CHANGELOG.md

@@ -9,10 +9,17 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Added
 - Storage of To: header in messages (likely breaks existing datastores)
+- Attachment list to [GET message
+  JSON](https://github.com/jhillyerd/inbucket/wiki/REST-GET-message)
 
 ### Fixed
-- We should no longer run out of file handles when dealing with a large number
+- No longer run out of file handles when dealing with a large number of
-  of recipients on a single message.
+  recipients for a single message.
+
+### Changed
+- Removed legacy `integral` theme, as most new features only in `bootstrap`
+- Removed old RESTful APIs, must use `/api/v1` base URI now
+- Allow increased local-part length of 128 chars for Mailgun
 
 [1.1.0] - 2016-09-03
 --------------------

smtpd/utils.go

@@ -182,8 +182,8 @@ LOOP:
 				inCharQuote = false
 			} else {
 				// End of local-part
-				if i > 63 {
+				if i > 128 {
-					return "", "", fmt.Errorf("Local part must not exceed 64 characters")
+					return "", "", fmt.Errorf("Local part must not exceed 128 characters")
 				}
 				if prev == '.' {
 					return "", "", fmt.Errorf("Local part cannot end with a period")

smtpd/utils_test.go

@@ -99,7 +99,8 @@ func TestValidateLocal(t *testing.T) {
 	}{
 		{"", false, "Empty local is not valid"},
 		{"a", true, "Single letter should be fine"},
-		{strings.Repeat("a", 65), false, "Only valid up to 64 characters"},
+		{strings.Repeat("a", 128), true, "Valid up to 128 characters"},
+		{strings.Repeat("a", 129), false, "Only valid up to 128 characters"},
 		{"FirstLast", true, "Mixed case permitted"},
 		{"user123", true, "Numbers permitted"},
 		{"a!#$%&'*+-/=?^_`{|}~", true, "Any of !#$%&'*+-/=?^_`{|}~ are permitted"},