aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/OpenSim (unfollow)
Commit message (Collapse)AuthorFilesLines
2012-09-27Fix very recently introduced race condition where a CreateNewItem outracing ↵Justin Clark-Casey (justincc)1-19/+18
an UploadAsset request could throw an exception because m_asset did not yet exist. This was accidentally introduced in 4fc0cfb This commit also consistently removes the AssetXferUploader when the transaction completes, no matter if it completed on asset upload or item operation. The amount of data being retained was small, since this was clothing/bodypart metadata in the asset rather than textures themselves.
2012-09-27Comment out old m_storeLocal from AssetXferUploader.Justin Clark-Casey (justincc)1-6/+6
This was only used if none of new item, update item or update task item had been set. But since all transactions go through these paths this old code is redundant.
2012-09-27Insert transaction ID into AssetXferUploader constructor rather than at ↵Justin Clark-Casey (justincc)2-36/+49
UploadAsset() to prevent item creation failure when NewInventoryItem thread reachs the object first. This was preventing the previous race condition fix in 4fc0cfb from actually working. This commit also removes some of the pointless transaction id checks - these conditions are already being enforced in AgentAssetsTransactions.
2012-09-27Move UDP update task item code to AssetXferUploader to match existing create ↵Justin Clark-Casey (justincc)3-50/+52
user item and update user item mechanisms This is done for consistency and to allow removal or some access methods that increase code complexity. However, this path has not been used for a long time, not even by LL 1.23 - viewers use caps http upload for this instead
2012-09-27Fix occasional race condition failure when creating new clothing/body parts ↵Justin Clark-Casey (justincc)3-110/+93
in the viewer or updating existing assets. On creating these items, the viewer sends a UDP AssetUploadRequest followed by a CreateInventoryItem. It was possible for the CreateInventoryItem/UpdateInventoryItem to occasionally outrace the AssetUploadRequest and fail to find an initialized Xfer object, at which point the item create would fail. So instead we always set up a Xfer object on either the asset or inventory item update request. This does not introduce a new race because code already exists to delay the item operation until the asset is uploaded if necessary (but this only worked if the xfer object already existed)
2012-09-26Use GridUser properly for foreign users.Diva Canto3-7/+32
2012-09-26Show the user's home url too on the console command.Diva Canto1-4/+3
2012-09-26Not really a change, just added a reminder to fix something here.Diva Canto1-0/+3
2012-09-25TOS module. WARNING: migration in GridUser table.Diva Canto5-5/+72
2012-09-25Oops, that should be inside the curly bracket.Diva Canto1-3/+3
2012-09-25Fix very recently introduced race condition where a CreateNewItem outracing ↵Justin Clark-Casey (justincc)1-19/+18
an UploadAsset request could throw an exception because m_asset did not yet exist. This was accidentally introduced in 4fc0cfb This commit also consistently removes the AssetXferUploader when the transaction completes, no matter if it completed on asset upload or item operation. The amount of data being retained was small, since this was clothing/bodypart metadata in the asset rather than textures themselves.
2012-09-25Comment out old m_storeLocal from AssetXferUploader.Justin Clark-Casey (justincc)1-6/+6
This was only used if none of new item, update item or update task item had been set. But since all transactions go through these paths this old code is redundant.
2012-09-25Insert transaction ID into AssetXferUploader constructor rather than at ↵Justin Clark-Casey (justincc)2-36/+49
UploadAsset() to prevent item creation failure when NewInventoryItem thread reachs the object first. This was preventing the previous race condition fix in 4fc0cfb from actually working. This commit also removes some of the pointless transaction id checks - these conditions are already being enforced in AgentAssetsTransactions.
2012-09-25Move UDP update task item code to AssetXferUploader to match existing create ↵Justin Clark-Casey (justincc)3-64/+52
user item and update user item mechanisms This is done for consistency and to allow removal or some access methods that increase code complexity. However, this path has not been used for a long time, not even by LL 1.23 - viewers use caps http upload for this instead
2012-09-25Fix occasional race condition failure when creating new clothing/body parts ↵Justin Clark-Casey (justincc)3-110/+94
in the viewer or updating existing assets. On creating these items, the viewer sends a UDP AssetUploadRequest followed by a CreateInventoryItem. It was possible for the CreateInventoryItem/UpdateInventoryItem to occasionally outrace the AssetUploadRequest and fail to find an initialized Xfer object, at which point the item create would fail. So instead we always set up a Xfer object on either the asset or inventory item update request. This does not introduce a new race because code already exists to delay the item operation until the asset is uploaded if necessary (but this only worked if the xfer object already existed)
2012-09-25Deleted extraneous debug messageDiva Canto1-2/+0
2012-09-24Documenting object-related eventsSignpostMarv1-0/+43
2012-09-24Documenting LSL script-related eventsSignpostMarv1-2/+147
2012-09-22HG Rez object: warn the user if the item or asset cannot be found.Diva Canto1-2/+7
2012-09-22Guard against inventory get failures.Diva Canto1-0/+3
2012-09-22Make BaseHttpServer throws say something useful.Diva Canto1-2/+2
2012-09-22HG 2.0: User Agent Service now can also control where the local users can ↵Diva Canto1-3/+100
go. Domain-name and user-level based. \o/
2012-09-21More HG 2.0: access control at the Gatekeeper. \o/Diva Canto1-8/+54
2012-09-21This fixes HG attachments' missing assets (textures, etc).Diva Canto2-10/+45
Also, further improvements on HGUuidGatherer: if the assets are already in this grid don't fetch them again.
2012-09-22Remove deprecated parameters for region_id.Michelle Argus1-74/+0
Remove deprecated parameters for region_id as anounced in january 2012.
2012-09-22Documenting non-LSL script-related eventsSignpostMarv1-0/+52
2012-09-21Moved the small HGUuidGatherer class to the file where its parent class is. ↵Diva Canto3-74/+38
No need to keep 2 separate files.
2012-09-22Fix llListFindList() returning no match when there is a match with a script ↵Justin Clark-Casey (justincc)2-8/+150
constant component in the source list. Adds regression test for this case. Based on http://opensimulator.org/mantis/view.php?id=6156 Thanks SignpostMarv.
2012-09-21Removed redundant asset fetches on HGAssetMapper. The UuidGatherer already ↵Diva Canto1-18/+24
downloads the assets, so we don't need to do it again...
2012-09-21Minor: may avoid crashes of sims that still don't have this configuration ↵Diva Canto2-1/+4
section.
2012-09-21HG 2.0: added asset import/export policies at the sim too.Diva Canto1-8/+15
2012-09-21Minor: change the return value of unsuccessful posts to string.Empty.Diva Canto2-2/+2
2012-09-20Improvement over last commit: refactor the asset permissions code, so that ↵Diva Canto2-58/+86
it can be used by both the HG Asset Service and the simulator. Also renamed the config vars to something more intuitive
2012-09-21minor: Make slow outgoing request log messages consistent with other log ↵Justin Clark-Casey (justincc)1-7/+7
messages
2012-09-21Rename UuidGather.m_assetCache to m_assetService. If HGUuidGatherer hasn't ↵Justin Clark-Casey (justincc)2-9/+6
been instantiated with an assetServerURL then call down to overriden UuidGatherer.GetAsset() instead of calling m_assetService.GetAsset() itself - these two codepaths are now identical.
2012-09-21Simplify UuidGatherer by performing asset fetch synchronously rather than ↵Justin Clark-Casey (justincc)1-40/+44
using the async call but waiting for completion anyway!
2012-09-21Comment out the long unused afaik HTTP agent handlers.Justin Clark-Casey (justincc)5-106/+112
As far as I know, this was only used by the IBM Rest modules, much of which has been commented out for a very long time now. Other similar code uses HTTP or stream handlers instead. So commenting this out to reduce code complexity and the need to make this facility consistent with the others where it may not be used anyway. If this facility is actually being used then please notify me or uncomment it if you are core.
2012-09-21Add request number counting to incoming HTTP requests in the same way that ↵Justin Clark-Casey (justincc)2-19/+42
this was already being done for outgoing HTTP requests. This allows us to associate debug logging messages with the right request. It also allows us to put a request number on 'long request' logging even if other debug logging is not enabled, which gives us some idea of whether every request is suffering this problem or only some. This is a separate internal number not associated with any incoming number in the opensim-request-id header, this will be clarified when logging of this incoming request number is re-enabled. This commit also adds port number to HTTP IN logging to allow us to distinguish between different request numbers on different ports.
2012-09-20More HG2.0: Added permission policies in HGAsset Service based on asset ↵Diva Canto2-18/+95
types. The policies are given in the config. This is only half of the story. The other half, pertaining to exports/imports made by the sim, will be done next.
2012-09-20Insert a new log level 4 for HTTP IN and HTTP OUT that will log how long the ↵Justin Clark-Casey (justincc)3-32/+87
request took. This is only printed if debug http level >= 4 and the request didn't take more than the time considered 'long', in which case the existing log message is printed. This displaces the previous log levels 4 and 5 which are now 5 and 6 respectively.
2012-09-20Add ability to turn on/off logging of outgoing HTTP requests flowing through ↵Justin Clark-Casey (justincc)3-35/+103
WebUtil. This is for debugging purposes. This is controlled via the "debug http" command which can already log incoming requests. This now gains a mandatory parameter of in, out or all to control what is logged. Log messages are also shortened and labelled and HTTP IN or HTTP OUT to be consistent with existing UDP PACKET IN and PACKET OUT messages.
2012-09-20Don't fail to create an IRC nick if nick randomization is disabled in the ↵Justin Clark-Casey (justincc)1-3/+3
IRC module. Patch from http://opensimulator.org/mantis/view.php?id=6293 Thanks Starflower.
2012-09-20Documentation of teleport-related eventsSignpostMarv1-0/+19
2012-09-20minor tweaks to existing comments for IDE goodnessSignpostMarv1-4/+7
2012-09-20Documentation of agent-related eventsSignpostMarv1-1/+34
2012-09-20Documentation of object-related eventsSignpostMarv1-1/+38
2012-09-20Documentation of OnPluginConsoleSignpostMarv1-0/+16
2012-09-20Make ResendAppearanceUpdates = true by default in [Appearance] in ↵Justin Clark-Casey (justincc)1-0/+2
OpenSimDefaults.ini. This resends appearance uuids to avatars in the scene once a minute. I have seen this help in the past resolve grey appearance problems where viewers have for unknown reasons sometimes ignored the packet. The overhead is very small since only the UUIDs are sent - the viewer then requests the texture only if it does not have it cached. This setting will not help with cloudy avatars which are usually due to the viewer not uploading baked texture data or uploading something that isn't valid JPEG2000
2012-09-19Correctly override and call base OpenSimTestCase.SetUp() method in ↵Justin Clark-Casey (justincc)3-6/+11
GridConnectorsTests and ArchiverTests. Remove unrelated compile warning from AttachmentsModuleTests.
2012-09-18Cleanup from prev. commitBlueWall1-2/+2
Make correct defaults to Phys/nonPhys prims to fix errors in prev. commit