[exim-dev] [Bug 2989] No error provided after invalid characters in local-part

2023-03-30 Thread admin--- via Exim-dev
https://bugs.exim.org/show_bug.cgi?id=2989

Jeremy Harris  changed:

   What|Removed |Added

   Target Milestone|Exim 4.96   |Exim 4.98+
   Severity|bug |wishlist
Version|4.96+   |4.95
   Priority|medium  |high

--- Comment #1 from Jeremy Harris  ---
Exim mostly doesn't have a need to interpret mail addresses in headers
(as opposed to the envelope), and will happily pass on such bad 822-level
data.

So this is a feature request.  Reclassifying from Bug to Wishlist.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##


Re: [exim-dev] Help debugging a tls smtp session

2023-03-30 Thread Andrew C Aitchison via Exim-dev


Thanks, Jeremy and Victor.

As is often the way, once I sent my plea,
the next search of the code found my problem.

I was setting the status value (done) to something other than "continue".
In fact I should not have touched it at all.

On Wed, 29 Mar 2023, Jeremy Harris via Exim-dev wrote:


On 28/03/2023 13:48, Andrew C Aitchison via Exim-dev wrote:


I'm attempting to write CLIENTID support for Exim.

My current attempt is available at
   https://www.aitchison.me.uk/exim/clientid.20230325.patch

I am using the exim test suite. Below is the significant part of
    runtest --keep 3752

When I do STARTTLS -> CLIENTID -> NOOP
the CLIENTID gives the correct response code but the next command
fails (it doesn't have to be a NOOP) in a way suggesting that I have
lost synchronization. Sometimes the TLS connection reports unexpected
commands in the input buffer (which I confirm to match what I expect
the script to send next).


Next step: add "-d+all" to the exim daemon line in the test script, and 
inspect the resulting "test-sterr-server" file.


--
Andrew C. Aitchison  Kendal, UK
   and...@aitchison.me.uk
--
## List details at https://lists.exim.org/mailman/listinfo/exim-dev Exim 
details at http://www.exim.org/ ##