[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: patch suggestion for GSMime.m
From: |
Richard Frith-Macdonald |
Subject: |
Re: patch suggestion for GSMime.m |
Date: |
Mon, 9 Sep 2013 09:31:23 +0100 |
On 8 Sep 2013, at 23:30, Pirmin Braun <pb@intars.de> wrote:
> Am Sun, 8 Sep 2013 21:36:31 +0100
> schrieb Richard Frith-Macdonald <richardfrithmacdonald@gmail.com> :
>
>> I would have thought the correct fix would be for the calling application to
>> use the existing methods to set the mime parser to be expecting http and/or
>> a different characterset, since this application should know the context
>> (ie what is being parsed and what form the browser has been told to upload
>> the data in).
>>
>>
>>
>
> the application using GSMime is the webserver-Library; in
> WebServerConnection.m it says:
> parser = [GSMimeParser new];
> [parser setIsHttp];
> so GSMimeParser will try NSISOLatin1StringEncoding, which obviously fails.
> Since _defaultEncoding is left unchanged at NSASCIIStringEncoding, no further
> attempts are made and it ends up with the NSLog(@"Bad header ... illegal
> characters in %@"
>
> so I thought, lets try all encodings. I think, the header is encoded in
> UTF-8. But will look at it with gdb now to be sure.
Sounds like there is something else going on ... if we are tryihng latin1 the
character conversion should never fail: since latin1 is an 8bit characterset
with all 256 available codepoints used, any data should represent a valid
string, so I guess the error message must be about something other than the
encoding.
- patch suggestion for GSMime.m, Pirmin Braun, 2013/09/06
- Re: patch suggestion for GSMime.m, Fred Kiefer, 2013/09/07
- Re: patch suggestion for GSMime.m, Pirmin Braun, 2013/09/07
- Re: patch suggestion for GSMime.m, Richard Frith-Macdonald, 2013/09/08
- Re: patch suggestion for GSMime.m, Pirmin Braun, 2013/09/08
- Re: patch suggestion for GSMime.m,
Richard Frith-Macdonald <=
- Re: patch suggestion for GSMime.m, Richard Frith-Macdonald, 2013/09/09
- Re: patch suggestion for GSMime.m, Pirmin Braun, 2013/09/09
- better patch suggestion for GSMime.m, Pirmin Braun, 2013/09/09
- Re: better patch suggestion for GSMime.m, Richard Frith-Macdonald, 2013/09/09
- Re: better patch suggestion for GSMime.m, Fred Kiefer, 2013/09/09