Skip to content
This repository has been archived by the owner. It is now read-only.

Kerberos dependancy prevents startup; obvious fix enclosed #310

Closed
macosforgebot opened this issue Aug 9, 2010 · 3 comments
Closed

Kerberos dependancy prevents startup; obvious fix enclosed #310

macosforgebot opened this issue Aug 9, 2010 · 3 comments

Comments

@macosforgebot
Copy link

@macosforgebot macosforgebot commented Aug 9, 2010

alexis@… originally submitted this as ticket:393

  • Cc: alexis@…, jas@…

Despite ticket 352 and changeset 4665, there is still a (new?) kerberos dependancy. In twistedcaldav/directory/principal.py, line 53, it says: from twistedcaldav.authkerb import NegotiateCredentials

authkerb.py will not run if kerberos isn't installed, as it will in turn try to import kerberos, which fails. There seemed little point in hacking on it since it has no purpose without kerberos, so I instead commented out the import in principal.py. This seems to work fine - the server appears to be fully working against iCal 10.6.

The "right" fix seems barely more complex - as with ticket #⁠352, stick it in a "try" block.

It would be nice to get this fix in soon, since server installs of Ubuntu 10.04 (and quite likely some other distros) don't come with kerberos installed.

Thanks.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Aug 9, 2010

alexis@… originally submitted this as comment:1:⁠ticket:393

  • Cc alexis@… added
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Oct 13, 2010

jas@… originally submitted this as comment:2:⁠ticket:393

  • Cc jas@… added
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Aug 1, 2011

@wsanchez originally submitted this as comment:3:⁠ticket:393

  • Radar deleted
  • Status changed from new to closed
  • Resolution changed from to Software changed

Fixed with #⁠356

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.