Opened 14 years ago

Closed 3 years ago

#4309 closed enhancement (wontfix)

[with patch, needs work] Kerberos authentification for SAGE notebook

Reported by: kkilger Owned by: kkilger
Priority: minor Milestone: sage-duplicate/invalid/wontfix
Component: notebook Keywords:
Cc: embray, jdemeyer, fbissey Merged in:
Authors: Reviewers: François Bissey
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description

This patch adds Kerberos authentification support for the SAGE notebook. It also adds the options krb_srv, krb_realm for the notebook() command.

This patch however depends on pykerberos which depends on kerberos which depends on a couple of other libraries all in all a little less size than SAGE itself and all not in SAGE :-). Nevertheless ...

Greetings, Kilian.

Attachments (4)

my.patch (4.2 KB) - added by kkilger 14 years ago.
sage-4309_1.patch (4.2 KB) - added by TimothyClemans 14 years ago.
sage-4309_2.patch (2.1 KB) - added by TimothyClemans 14 years ago.
sage-4309_3.patch (949 bytes) - added by TimothyClemans 14 years ago.

Download all attachments as: .zip

Change History (19)

Changed 14 years ago by kkilger

comment:1 Changed 14 years ago by kkilger

  • Summary changed from Kerberos authentification for SAGE notebook to [with patch, needs review] Kerberos authentification for SAGE notebook

comment:2 Changed 14 years ago by mabshoff

  • Milestone set to sage-3.2

comment:3 Changed 14 years ago by TimothyClemans

This doesn't apply in sage-3.2.alpha1

Changed 14 years ago by TimothyClemans

comment:4 Changed 14 years ago by TimothyClemans

  • Summary changed from [with patch, needs review] Kerberos authentification for SAGE notebook to [with patch, needs work] Kerberos authentification for SAGE notebook

Kerberos shouldn't be required to use the notebook. Make Kerberos authentication optional.

comment:5 Changed 14 years ago by TimothyClemans

my.patch is replaced by sage-3950_1.patch and applies in sage-3.2.alpha1

Changed 14 years ago by TimothyClemans

comment:6 Changed 14 years ago by TimothyClemans

  • Summary changed from [with patch, needs work] Kerberos authentification for SAGE notebook to [with patch, needs review] Kerberos authentification for SAGE notebook

OK now Kerberos isn't required.

Changed 14 years ago by TimothyClemans

comment:7 Changed 14 years ago by TimothyClemans

  • Component changed from user interface to notebook
  • Owner changed from was to kkilger

comment:8 Changed 13 years ago by was

  • Summary changed from [with patch, needs review] Kerberos authentification for SAGE notebook to [with patch, needs work] Kerberos authentification for SAGE notebook

Hi,

I like the design of this patch and how the kerberos stuff is optional.

Unfortunately, this is hard to referee if you don't explain to me how to install pykerberos. Also, the patch itself should contain code that says "hey, you need kerberos, and here's how to install it" when the user turns on kerberos via an option to notebook, but

import kerberos

fails. That could be either a paragraph of text, an optional spkg to install, or a link to a wiki page that describes what to do.

OK?

Thanks!

comment:9 Changed 9 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:10 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:11 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:12 Changed 8 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:13 Changed 3 years ago by chapoton

  • Cc embray jdemeyer fbissey added
  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Report Upstream set to N/A
  • Status changed from needs_work to needs_review

notebook is deprecated, so this can probably be closed

comment:14 Changed 3 years ago by fbissey

  • Reviewers set to François Bissey
  • Status changed from needs_review to positive_review

Yes.

comment:15 Changed 3 years ago by chapoton

  • Resolution set to wontfix
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.