Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add krb5.aname_to_localname() #19

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Commits on Oct 31, 2022

  1. Have krb5.init_context() raise an exception on error.

    init_context() currently just returns a null pointer if the C routine
    fails, which is unlike the norm in this module and I assume is an
    oversight. Typically this will fail if there's a syntax error in the
    libkrb5 configuration (/etc/krb5.conf, or whatever is read). A program
    I wrote segfaulted in this situation, because I then passed the null
    context to other routines. With this change, we get instead:
    
    In [2]: ctx = krb5.init_context()
    ---------------------------------------------------------------------------
    Krb5Error                                 Traceback (most recent call last)
    ...
    Krb5Error: Included profile file could not be read -1429577697
    pseudometric committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    cda1e05 View commit details
    Browse the repository at this point in the history
  2. add krb5_aname_to_localname()

    This is part of the MIT Kerberos "localauth" interface:
    
      https://web.mit.edu/kerberos/krb5-devel/doc/plugindev/localauth.html
    
    ... so I used that as the "extension" grouping name. There is also a
    krb5_kuserok() in this interface, that might be added.
    pseudometric committed Oct 31, 2022
    Configuration menu
    Copy the full SHA
    d141410 View commit details
    Browse the repository at this point in the history

Commits on Nov 16, 2022

  1. Revert "Have krb5.init_context() raise an exception on error."

    I'm moving this to its own PR; it's unrelated to the
    krb5_aname_to_localname() addition.
    
    This reverts commit cda1e05.
    pseudometric committed Nov 16, 2022
    Configuration menu
    Copy the full SHA
    da355d7 View commit details
    Browse the repository at this point in the history