[Pharo-project] 1.3-Core

Alexandre Bergel alexandre.bergel at me.com
Fri May 13 09:24:38 EDT 2011


Hi Sven,

First of all, thanks for caring about the problem I encounter.

> OK, here is a quick fix/patch:
> 
> ZnRequestLine>>writeOn: stream
> 	| writeStream |
> 	writeStream := ZnBivalentWriteStream on: stream.
> 	writeStream nextPutAll: self method; space.
> 	ZnNetworkingUtils default isProxySet 
> 		ifTrue: [ self uri printOn: writeStream ]  
> 		ifFalse: [ self uri printPathQueryFragmentOn: writeStream ].
> 	writeStream space; nextPutAll: self version; nextPutAll: String crlf 

#default is not defined on 'ZnNetworkingUtils class'. I removed the call to #default, and it works. 
Printing the expression gave me a ZnResponse(200 OK text/html;charset=utf-8 113B)
which looks good

> The problem was that for an HTTP request through a proxy, the url in the request line must be absolute.
> 
> Let me know if this works for you.

Yes it works.

> There is another problem as well: access to, at least localhost, should not go through the proxy.
> I am fixing that right now.
> I want you to be able to run all Zn unit tests with a proxy set, but half of them are accessing localhost.
> Anyway, the full fixes will be committed to the Zn repos first, and will have to be migrated to Pharo next.

I use a core 13197. Running Zinc tests gives me:
146 run, 118 passes, 0 expected failures, 28 failures, 0 errors, 0 unexpected passes

Cheers,
Alexandre
-- 
_,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:
Alexandre Bergel  http://www.bergel.eu
^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;._,.;:~^~:;.









More information about the Pharo-dev mailing list