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 10 years ago

Closed 9 years ago

Last modified 9 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 10 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 9 years ago by jmoore

  • Component changed from Services to Security

comment:3 Changed 9 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 9 years ago by jmoore

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

comment:5 Changed 9 years ago by jmoore

  • Remaining Time set to 0.5

comment:6 Changed 9 years ago by jmoore

  • Description modified (diff)

comment:7 Changed 9 years ago by jmoore

  • Status changed from new to assigned

comment:8 Changed 9 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 9 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 9 years ago by jmoore

See also #1731

comment:11 Changed 9 years ago by jmoore

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

comment:12 Changed 9 years ago by jmoore

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

comment:13 Changed 9 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.89446 sec.)

We're Hiring!