Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0007694opensim[REGION] Script Functionspublic2015-08-20 18:172015-08-20 18:36
ReporterChris Weymann 
Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Statuspatch includedResolutionopen 
PlatformLinuxOSDebianOS Version8
Product Versionmaster (dev code) 
Target VersionFixed in Version 
Summary0007694: llrequestpermissions and llhttprequest dont get object owner name then the owner is an HG user
DescriptionI have found again two things in the script funktions llrequestpermissions and llhttprequest who are affect the name resolution.

In both cases is the object owner name used. Then this owner is an HG user its impossible.

At last i have remove the unused method resolveName.

I have split this in 3 patches.
TagsNo tags attached.
Git Revision or version number
Run ModeStandalone (1 Region)
Physics EngineBulletSim
Script Engine
EnvironmentMono / Linux64
Mono Version4.0.1
ViewerFirestorm 4.6.9
Attached Filespatch file icon llhttprequest.patch [^] (1,594 bytes) 2015-08-20 18:17 [Show Content]
patch file icon llrequestpermissions .patch [^] (1,433 bytes) 2015-08-20 18:17 [Show Content]
patch file icon resolveName.patch [^] (2,144 bytes) 2015-08-20 18:18 [Show Content]

- Relationships

-  Notes
(0029348)
melanie (administrator)
2015-08-20 18:24

HARD -1

Change ResolveName to check HG users but please note that ResolveName is ALSO capable of resolving object names and group names. You are completely removing this functionality without replacement. Irregardless of it not currently being used, thought went into the design of that method.

Change ResolveName to use the UserManagementModule instead of the UserAccountService but do NOT remove it. It may be needed by other APIs to resolve object or group names and that functionality is too valuable to remove.
(0029349)
Chris Weymann (reporter)
2015-08-20 18:32
edited on: 2015-08-20 18:32

Ok, im was not sure that i shold make with a funktion who are not used anymore.
If you believe this shold keep in place then dont apply the patch who remove it.
For this i have make it into a seperate patch.

(0029350)
melanie (administrator)
2015-08-20 18:36

No. What I want is for you to CHANGE ResolveName to use the new service. Your patch removes functionality while adding code duplication.
Not acceptable in this way.
Just give me one patch to CHANGE ResolveName. Do NOT remove it. Do NOT change llRequestPermissions and do NOT change llHTTPRequest. Simply change ResolveName.

- Issue History
Date Modified Username Field Change
2015-08-20 18:17 Chris Weymann New Issue
2015-08-20 18:17 Chris Weymann File Added: llhttprequest.patch
2015-08-20 18:17 Chris Weymann File Added: llrequestpermissions .patch
2015-08-20 18:18 Chris Weymann File Added: resolveName.patch
2015-08-20 18:18 Chris Weymann Status new => patch included
2015-08-20 18:24 melanie Note Added: 0029348
2015-08-20 18:32 Chris Weymann Note Added: 0029349
2015-08-20 18:32 Chris Weymann Note Edited: 0029349 View Revisions
2015-08-20 18:36 melanie Note Added: 0029350


Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker