Task #1704 (closed)

Opened 9 months ago

Last modified 4 months ago

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

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

Description (last modified by jmoore) (diff)

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

References

Referenced by:
  • ← User Story (#1434): Re-enable group permissions support

Change History

Changed 9 months 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----'

Changed 7 months ago by jmoore

  • component changed from Services to Security

Changed 6 months ago by jmoore

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

Changed 6 months ago by jmoore

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

Changed 6 months ago by jmoore

  • remaining_time set to 0.5

Changed 6 months ago by jmoore

  • description modified (diff)

Changed 5 months ago by jmoore

  • status changed from new to assigned

Changed 5 months 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.

Changed 5 months ago by jmoore

  • status changed from assigned to closed
  • resolution set to fixed
  • remaining_time changed from 0.5 to 0

(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.

Changed 5 months ago by jmoore

See also #1731

Changed 5 months ago by jmoore

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

Changed 5 months ago by jmoore

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

Changed 4 months 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.2.1-PRO © 2008-2009 agile42 all rights reserved (this page was served in: 0.309317 sec.)