Projects
Wiki     Timeline     Roadmap     Browse Source     View Tickets     New Ticket     Search

Ticket #115 (closed Defect: Software changed)

Opened 6 years ago

Last modified 6 months ago

principalIsGroupMember doesn't do the right thing with the new directory service and principal APIs

Reported by: dreid@… Owned by: cdaboo@…
Priority: 1: Blocker Milestone: CalendarServer-1.0
Component: Calendar Server Severity: Serious
Keywords: Cc: kurt.leubner@…
Port:

Description

It tries to find pincipal1 (which is a string principal URI) in a set of DirectoryPrincipalResources. In the future principal1 which comes from request.authzPrincipal will actually be a IDAVPrincipalResource, so it can just lookup the principal for the group which will and do in on the groupMembers set

For now however it should just lookup both principals and and do the comparison.

Also it's method signature is not very helpful, principal1, and principal2 are not very good variable names principal and group would be better.

Change History

comment:1 Changed 6 years ago by wsanchez@…

  • Status changed from new to assigned
  • Milestone set to Preview 1

comment:2 Changed 6 years ago by wsanchez@…

  • Status changed from assigned to new
  • Owner changed from wsanchez@… to cdaboo@…

comment:3 Changed 6 years ago by wsanchez@…

  • Status changed from new to closed
  • Resolution set to fixed

ok in trunk

comment:4 Changed 6 years ago by wsanchez@…

  • Milestone changed from Preview 1 to Version One

Milestone Preview 1 deleted

comment:5 Changed 6 months ago by kurt.leubner@…

  • Cc kurt.leubner@… added

Cc Me!

comment:6 Changed 6 months ago by kurt.leubner@…

  • Cc kurt.leubner@… removed

Cc Me!

comment:7 Changed 6 months ago by kurt.leubner@…

  • Cc kurt.leubner@… added

Cc Me!

Note: See TracTickets for help on using tickets.