Projects
Wiki     Timeline     Roadmap     Browse Source     View Tickets     New Ticket     Search

Ticket #305 (closed Defect: Software changed)

Opened 5 years ago

Last modified 9 months ago

Apache directory service types broken?

Reported by: msantos@… Owned by: wsanchez@…
Priority: 3: Important Milestone: Later
Component: Calendar Server Severity: Crash/data loss
Keywords: Cc: kurt.leubner@…
Port:

Description (last modified by agx@…) (diff)

It appears that the Apache directory service types are broken. Code in udpateDirectoryService() in config.py seems to be trying to validate the directory service parameters from the config file against the known keywords, but there is no entry for them in serviceDefaultParams[], so the code takes a KeyError exception at this line:

if param not in serviceDefaultParams[dsType]:

because dsType is twistedcaldav.directory.apache.DigestDirectoryService, but the list in config.py only has entries for:

twistedcaldav.directory.xmlfile.XMLDirectoryService twistedcaldav.directory.appleopendirectory.OpenDirectoryService

Maybe that list is supposed to be updated somewhere else (from apache.py?), but I couldn't find any such place.

Change History

comment:1 Changed 5 years ago by wsanchez@…

  • Status changed from new to assigned

See http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=503727 for a Debian bug on this and a patch.

comment:2 Changed 5 years ago by agx@…

  • Description modified (diff)

Hi Winfredo, the patch makes sense to me. Can it be applied?

-- Guido

comment:3 in reply to: ↑ description Changed 5 years ago by agx@…

  • Description modified (diff)

Replying to msantos@…:

Hi Winfredo, the patch in the BTS

comment:4 Changed 5 years ago by agx@…

Whoops. I didn't mean to change the description, just wanted to add a comment.

Hi Winfredo, the patch makes sense to me. Can it be applied?

Sorry for the noise.

comment:5 Changed 5 years ago by wsanchez@…

OK, I applied these and some other work that seemed obviously lacking. The files appear to read in correctly, but a couple of things (at least) are still broken:

  • the basic verifyCredentials bits still don't work for me; I wasn't able to auth
  • the digest verifyCredentials bits are unimplemented
  • the digest directory service doesn't filter out records with a non-matching realm (realm seems irrelevant to basic)

comment:6 Changed 4 years ago by wsanchez@…

  • Priority changed from 5: Not set to 3: Important
  • Milestone set to Later

comment:7 Changed 3 years ago by wsanchez@…

  • Status changed from assigned to closed
  • Resolution set to Software changed

This functionality was removed for 3.0

comment:8 Changed 9 months ago by kurt.leubner@…

  • Cc kurt.leubner@… added

Cc Me!

Note: See TracTickets for help on using tickets.