Hi Jason, +1 for both! Waiting for the patches ... :-)
Thanks -Vincent > -----Original Message----- > From: Robertson, Jason [mailto:[EMAIL PROTECTED] > Sent: 11 February 2003 03:26 > To: 'Cactus Users List' > Subject: WebRequest and header values > > The current WebRequest object only has an "addHeader" method, and we've > come > across a need for a "setHeader" method and I was wondering if there was > any > specific reason something like this isn't there. The difference would be > that in setHeader it would replace any existing values with the new value. > > The issue has appeared because some of my team members implemented a test > case that in 29 of the 34 test methods has a common configuration that > they > put into the begin() method. Then, in those 5 other tests, they need > different configuration options which they put into beginXXX() methods > (this > way they didn't have to write 34 beginXXX() methods). However, the > configuration involves setting header values and since there's no way to > override the existing headers the begin() configuration put there they > can't > be replaced in beginXXX(). > > Also, it was suggested that instead of using a Vector for the header > values > it should be an ArrayList (or some other unsynchronized collection) so > that > you don't pay the slight but unnecessary synchronization penalty. > > I have the guy rewriting WebRequest with these changes, I'll submit it > when > he has it ready. I just wanted to toss these comments out before he gets > too > far. > > Jason > > --------------------------------------------------------------------- > 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]
