[incubator-ponymail-foal] 01/02: Caller provides properly encoded URI

[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-foal.git

commit b4058c500adb650ebad05c771cc17fbec508ac78
Author: Sebb <se...@apache.org>
AuthorDate: Fri Jan 7 23:58:42 2022 +0000

    Caller provides properly encoded URI
    
    No need to see if '+' meant space or '+'
---
 server/endpoints/thread.py | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/server/endpoints/thread.py b/server/endpoints/thread.py
index 13adf9f..6fd045a 100644
--- a/server/endpoints/thread.py
+++ b/server/endpoints/thread.py
@@ -28,6 +28,8 @@ async def process(
 ) -> typing.Optional[dict]:
     mailid = indata.get("id", "")
     listid = indata.get("listid", "")
+    print(mailid)
+    print(listid)
 
     # lookup by message id must always include a list id for disambiguation
     if listid:
@@ -35,12 +37,6 @@ async def process(
     else: # Else assume permalink and look up the email based on that
         email = await plugins.messages.get_email(session, permalink=mailid)
 
-    # The id is passed via the path thread/id
-    # This means that + is converted into space
-    # So we need to try both space and '+', and hope no msg ids contain both
-    if not email and ' ' in mailid: # only try again if we need to
-        mailid = mailid.replace(" ", "+")
-        email = await plugins.messages.get_email(session, messageid=mailid, listid=listid)
     if not email:
         return None
     if indata.get("find_parent"):