MantisBT - opensim
View Issue Details
0005081opensim[REGION] OpenSim Corepublic2010-10-08 08:582010-10-27 20:37
thickbrick 
thickbrick 
normalmajoralways
closedfixed 
 
 
76fd4cf
Grid (1 Region per Sim)
BasicPhysics
Unknown
None
0005081: HTTP GET requests to the GetTexture cap return one less byte than requested.
Doing
curl -r 0-599 <GetTexture cap url>?texture_id=<texture UUID> > file.j2c

Will result in a 599 bytes file, and the following http headers:
HTTP/1.1 206 PartialContent
Date: Fri, 08 Oct 2010 08:02:20 GMT
Content-Length: 599
Content-Type: text/html;charset=UTF-8
Server: Tiny WebServer
Connection: close

This should get 600 bytes, not 599. On other http servers (as well as SL sims) the file will be 600 bytes, as expected.

This was observed on 3rdRock grid, on OpenSim 0.6.9 (Post_Fixes) d6f14d5-r/13679 (Win/.NET)
And on OSGrid on OpenSim 0.6.9 (Post_Fixes) 76fd4cf: 2010-09-30 15:43:56 +0100 (Unix/Mono)
texture
Issue History
2010-10-08 08:58thickbrickNew Issue
2010-10-08 08:58thickbrickGit Revision => 76fd4cf
2010-10-08 08:58thickbrickSVN Revision => 0
2010-10-08 08:58thickbrickRun Mode => Grid (1 Region per Sim)
2010-10-08 08:58thickbrickPhysics Engine => BasicPhysics
2010-10-08 08:58thickbrickEnvironment => Unknown
2010-10-08 08:58thickbrickMono Version => None
2010-10-08 09:03thickbrickSummaryHTTP requests to the GetTexture cap return one less byte than requested. => HTTP GET requests to the GetTexture cap return one less byte than requested.
2010-10-08 09:03thickbrickDescription Updated
2010-10-08 16:54thickbrickNote Added: 0016939
2010-10-08 16:54thickbrickStatusnew => resolved
2010-10-08 16:54thickbrickResolutionopen => fixed
2010-10-08 16:54thickbrickAssigned To => thickbrick
2010-10-09 05:38thickbrickTag Attached: texture
2010-10-09 05:42thickbrickNote Added: 0016944
2010-10-09 05:42thickbrickStatusresolved => confirmed
2010-10-09 05:43thickbrickStatusconfirmed => resolved
2010-10-27 20:37chi11kenStatusresolved => closed

Notes
(0016939)
thickbrick   
2010-10-08 16:54   
Resolving this, as it was fixed in commit 5ef2da39d81c038c087b493330497856ed18325d :

* Fixing length calculations for HTTP texture downloads (the end byte is inclusive in Range: headers)

John Hurliman (author)
September 13, 2010

I also verified it's fixed in the binary Diva Distro 0.7.0.2 (OpenSim 0.7.0.2 Post_Fixes)
(0016944)
thickbrick   
2010-10-09 05:42   
Re-opening to add a note that I was wrong above: This fix is not in 0.7.0.2 yet. (I hope changing to "Confirmed" and re-resolving doesn't cause too big a mess)