Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: ripMIME: general

More ripMIME beta updates

 

 

ripMIME general RSS feed   Index | Next | Previous | View Threaded


pldaniels at pldaniels

Jan 26, 2003, 6:10 AM

Post #1 of 3 (952 views)
Permalink
More ripMIME beta updates

More updates, this time dealing with the handling of wrapped headers, multiple ISO encoded blocks and now the creation of a new module (libmime-decoders) to which all decoding routines (currently short BASE64 and quoted-printable decoding) will be placed.

I'm moving more the decoding routines into their own module because I need to eventually be able to call them from the mime_headers so I can decode other ISO encoded components of the headers.

Once again, the updated code is available at:

http://www.pldaniels.com/ripmime/ripmime-1.2.16.23.tar.gz

Sorry there have been so many changes ... I'm just in the mood to be doing it all ( for a change, I've been really slow on coding the last 2 months ).


Here's the extract from the CHANGELOG
------------
Sun Jan 24 2003
- PLD: in MIME_decode_filename(), removed limitation of filenames only having
a space character and lengths greater than two (2).

- PLD: Corrected header unfolding in MIMEH_read_headers()
This rectifies the situation where some filenames are cropped because they
were spread over two lines in the headers.

- PLD: Moved ISO decoding functions and QuotedPrintable functions into a new
module called 'libmime-decoders'.

- PLD: Initialised encoding_type in the ISO decoding routines in order to
silence some compilers about a 'usage unitilised' warning.

- PLD: Corrected situation where an attempt to extract a file name would
be misled due to a substring 'name' appearing anywhere within headers.

Narrowed down the conditions required in order for a 'name' word to appear.
This corrects the issue as reported by Chris Hine

- PLD: I've sorted out the Quoted-Printable decoding
issues. Who would have thought such a relatively simple
decoding process could cause so much pain. As a consequence
some of the prior 'patches' have now been removed as they are
no longer needed. ( removed patches: Mirko Buffoni )

------------

--
Paul L Daniels http://www.pldaniels.com
Linux/Unix systems Internet Development
ICQ#103642862,AOL:cinflex,IRC:inflex
A.B.N. 19 500 721 806


cph at oce

Jan 26, 2003, 6:39 AM

Post #2 of 3 (898 views)
Permalink
RE: More ripMIME beta updates [In reply to]

Paul,

Just downloaded and tested the new version, and its looking good.

I have just discovered another minor problem though.
I am using ripmime with qmail-scanner, which doesn't like any output on
stdout, and there are a couple of places in mime.c where it writes to
stdout, even if you haven't turned on debugging.
I have included a patch below to correct this.
I don't think I've screwed anything up, but let me know if I have.

Chris

--- mime.c.orig Sun Jan 26 13:03:18 2003
+++ mime.c Sun Jan 26 13:30:37 2003
@@ -1569,7 +1569,7 @@
{
if (strcasecmp(hinfo->uudec_name,"winmail.dat")==0)
{
- fprintf(stdout,"Decoding TNEF format\n");
+ if (MIME_DNORMAL) fprintf(stdout,"Decoding
TNEF format\n");
snprintf(hinfo->filename, 128, "%s",
hinfo->uudec_name);
MIME_decode_TNEF( unpackdir, hinfo, keep);
}
@@ -1715,7 +1715,7 @@

if (strncasecmp(hinfo->uudec_name,"winmail.dat",11)==0)
{
- fprintf(stdout,"Decoding TNEF format\n");
+ if (MIME_DNORMAL) fprintf(stdout,"Decoding TNEF
format\n");
snprintf(hinfo->filename, 128, "%s",
hinfo->uudec_name);
MIME_decode_TNEF( unpackdir, hinfo, keep );
}
@@ -2273,7 +2273,7 @@
// If the calling program requested verbosity, then indicate that
we're decoding
// the file here

- if ((keep)&&(glb.verbosity)) fprintf(stdout,"Decoding %s\n",
hinfo->filename);
+ if (((keep)&&(glb.verbosity))&&(MIME_DNORMAL))
fprintf(stdout,"Decoding %s\n", hinfo->filename);


// Select the decoding method based on the content transfer encoding


pldaniels at pldaniels

Jan 26, 2003, 8:29 AM

Post #3 of 3 (906 views)
Permalink
Re: More ripMIME beta updates [In reply to]

> I don't think I've screwed anything up, but let me know if I have.
>

Only one that I didn't apply, which was :


> - if ((keep)&&(glb.verbosity)) fprintf(stdout,"Decoding %s\n", hinfo->filename);

As this should be displayed when verbosity is selected.

There was a error in some of todays earlier versions which still had the verbosity hard-fixed on '1' ( in MIME_init() ) - This has been rectified.

I've also cleaned up a couple of other possible messages which could upset qmail in MIME_headers.

Looking over the whole system of the mime and MIME_headers modules, I see that my next job should be to clean up a lot of the message/logging choices.

Regards.

http://www.pldaniels.com/ripmime/ripmime-1.2.16.23.tar.gz


--
Paul L Daniels http://www.pldaniels.com
Linux/Unix systems Internet Development
ICQ#103642862,AOL:cinflex,IRC:inflex
A.B.N. 19 500 721 806

ripMIME general RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.