[incubator-ponymail] branch master updated: Bug: does not make sense to allow empty domain name in LID

[se...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-ponymail.git


The following commit(s) were added to refs/heads/master by this push:
     new a0e6980  Bug: does not make sense to allow empty domain name in LID
a0e6980 is described below

commit a0e69803086eca262381b57a5012f628353d1769
Author: Sebb <se...@apache.org>
AuthorDate: Tue Feb 20 18:39:31 2018 +0000

    Bug: does not make sense to allow empty domain name in LID
    
    This fixes #434
---
 CHANGELOG.md         | 1 +
 site/api/lib/aaa.lua | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 5f97ef1..28012a8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,6 @@
 ## Changes in 0.11-SNAPSHOT
 - Bug: Indentation in mail content (leading white-space) not shown (#432)
+- Bug: does not make sense to allow empty domain name in LID (#434)
 
 ## CHANGES in 0.10:
 - Bug: Use correct dependency order to fix OAuth login problem (#431)
diff --git a/site/api/lib/aaa.lua b/site/api/lib/aaa.lua
index 5d58db2..73bbd41 100644
--- a/site/api/lib/aaa.lua
+++ b/site/api/lib/aaa.lua
@@ -101,7 +101,7 @@ end
   The domain is the rest. The full lid is the whole input, without < and >.
 ]]
 local function parseLid(lid)
-    return lid:match("^<(([^.]+)%.(.-))>$")
+    return lid:match("^<(([^.]+)%.(.+))>$")
 end
 
 

-- 
To stop receiving notification emails like this one, please contact
sebb@apache.org.