Thanks for your effort, unfortunately attachments don't seem to come through on the list. Could you create a new bug in bugzilla for this and attach the file to that?
Thanks,
Adrian Sutton
On Saturday, June 21, 2003, at 12:57 PM, Eric M Devlin wrote:
Hey Mike, Adrian & Oleg
Got it. Thanks for the kick in the right direction. I wasn't seeing
addPart. Please find attached the contrib file. Hope it helps someone
else. I tried to make it as apacheish as possible if it needs any changes
just let me know. Thanks to everyone working on httpclient. It's been a
really nice find. I'm using to at work, but I've gotten more out of it at
home.
Eric
-----Original Message----- From: Michael Becke [mailto:[EMAIL PROTECTED] Sent: Thursday, June 19, 2003 11:01 PM To: Commons HttpClient Project Subject: Re: PATCH org.apache.commons.httpclient.methods.multipart.FilePart
Eric,
You want to create the FilePart manually and then add it to the MultipartPostMethod. Something like:
MultipartPostMethod mpm = new MultipartPostMethod( "http://localhost:8080/something"; ); FilePart part = new FilePart(name, file, contentType, charset); mpm.addPart(part);
Mike
On Thursday, June 19, 2003, at 10:23 PM, Eric M Devlin wrote:
Hey Adrian, Ok, but what about what something below?
MultipartPostMethod mpm = new MultipartPostMethod( "http://localhost:8080"; + "/something" ); mpm.addParameter( "A", new File( "/usr/dev/images/add.gif" ) );
How do I set the content type for the file? It seems either the addParameter needs to return FilePart which would have to have setters provided or an overloaded version of addParameter which accepted the content type and charset.
//MultipartPostMethod public void addParameter(String parameterName, File parameterFile, String contentType, String charset )
-- or --
//MultipartPostMethod
public FilePart addParameter(String parameterName, File parameterFile )
//FilePart public void setContentType( String contentType ) public void setCharset( String charset )
I've got the code pulled into class but without a place to apply it's going to be pretty useless. ;-> I think this is why I was putting the contentType determination in the FilePart class it self. Just let me know.
Eric
-----Original Message----- From: Adrian Sutton [mailto:[EMAIL PROTECTED] Sent: Wednesday, June 18, 2003 11:10 PM To: Commons HttpClient Project Subject: Re: PATCH org.apache.commons.httpclient.methods.multipart.FilePart
Hi Eric, This isn't really something that should be included directly into HttpClient as HttpClient isn't intended to care about the actual content it sends and receives but just takes care of the actual HTTP protocol side of things. Adding auto-mime type detection would mean we'd also have to provide a way to configure the default mime-types etc, in other words it opens a whole can of worms.
However, this would be an excellent submission to the HttpClient contrib package, particularly if we refactor it so that instead of being a patch it's a complete class that extends FilePart to add the functionality, then it could easily be used without any changes to HttpClient. Would you be happy with that course of action? If so, would you like to adapt the patch into a standalone class yourself or would you like me to take a crack at it? I don't mind either way.
Thanks a lot for the contribution, it will definitely be useful to a number of people.
Regards,
Adrian Sutton.
On Thursday, June 19, 2003, at 12:46 PM, Eric M Devlin wrote:
Hey,
This is a patch which will determine the content type if null based
on
file extension. I used the file extension mapping from
$TOMCAT_HOME/conf/web.xml. As a side note, I'm having trouble sending
gif
files. Any thoughts or a kick in the right direction would be
helpful.
Thanks and Hope It Helps
Eric
<contentTypeByExtension.txt>----------------------------------------- -
-
--------------------------
To unsubscribe, e-mail:
[EMAIL PROTECTED]
For additional commands, e-mail:
[EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
