Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #1704 (closed)

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

BUG: ISession.update is broken wrt defaultPermissions and perhaps other

Reported by: jamoore Owned by: jamoore
Priority: critical Milestone: OMERO-Beta4.2
Component: Security Version: 4.1
Keywords: n.a. Cc: carlos@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2010-03-19 (5)

Description (last modified by jmoore)

Looks like the call to setDefaultPermissions in the session candidate is being ignored and then overriden by the original session value.

Be sure to see the SpecialObjectsPermTest commits from #1794

Change History (13)

comment:1 Changed 14 years ago by jmoore 

In [22]: ss = s.getSessionService()

In [23]: sess = ss.getSession(s.ice_getIdentity().name)

In [24]: sess.defaultPermissions.val  
Out[24]: 'rw----'

In [25]: sess.defaultPermissions = omero.rtypes.rstring("rwrw--")

...

In [27]: sess = ss.updateSession(sess)

In [28]: sess.defaultPermissions.val
Out[28]: 'rw----'

comment:2 Changed 14 years ago by jmoore

  • Component changed from Services to Security

comment:3 Changed 14 years ago by jmoore

  • Summary changed from ISession.update is broken wrt defaultPermissions and perhaps other to BUG: ISession.update is broken wrt defaultPermissions and perhaps other
  • Type changed from Bug to Task

comment:4 Changed 14 years ago by jmoore

  • Sprint set to 2010-03-19 (5)

comment:5 Changed 14 years ago by jmoore

  • Remaining Time set to 0.5

comment:6 Changed 14 years ago by jmoore

  • Description modified (diff)

comment:7 Changed 14 years ago by jmoore

  • Status changed from new to assigned

comment:8 Changed 14 years ago by jmoore

(In [6453]) see #1704 - removing updateSession

After a discussion with the team, the only remaining use
of updateSession was the setting of the active group from
4.1. This has now been replaced by setSecurityContext
and can be removed.

comment:9 Changed 14 years ago by jmoore

  • Remaining Time changed from 0.5 to 0
  • Resolution set to fixed
  • Status changed from assigned to closed

(In [6454]) fix #1704 - removing default permissions

defaultPermissions and all umask support is also no longer
needed due to the restrictions of #1434 - all objects in a group
have the same permissions.

comment:10 Changed 14 years ago by jmoore

See also #1731

comment:11 Changed 14 years ago by jmoore

(In [6570]) see #1704 - removing umask argument from createUserSession

comment:12 Changed 14 years ago by jmoore

(In [6586]) see #1704 - Removing defaultpermissions from data.vm

comment:13 Changed 14 years ago by jmoore

(In [6829]) see #1704 - Also removing umask from System.ice

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.67774 sec.)

© 2000-2014 University of Dundee & Open Microscopy Environment. Creative Commons Attribution 3.0 Unported License
OME source code is available under the GNU General public license or through commercial license from Glencoe Software

We're Hiring!