Skip to content

Improve error handling with Kerberos errors #361

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

Closed
wooldridge opened this issue Feb 4, 2017 · 1 comment
Closed

Improve error handling with Kerberos errors #361

wooldridge opened this issue Feb 4, 2017 · 1 comment

Comments

@wooldridge
Copy link
Contributor

The following is insufficient in the case of a Kerberos error under the hood:

kerberos.authGSSClientInit(uri, 0, function(err, ctx) {
    if (err) {
      operation.errorListener('kerberos initialization failed at '+uri);
    }
    operation.logger.debug('kerberos initialized at '+uri);
    kerberos.authGSSClientStep(ctx, "", function (err) {
      if (err) {
        operation.errorListener('kerberos credentials failed');
      }

Need to at least display the err object. This was apparent in #343 (comment) when I had to console.log() the authGSSClientStep err object to figure out what was going on.

@wooldridge wooldridge self-assigned this Feb 4, 2017
@wooldridge wooldridge added this to the 9.0-1 milestone Feb 4, 2017
@naliniprakash naliniprakash modified the milestones: 9.0-2, 9.0-1 Mar 6, 2017
@wooldridge wooldridge added minor and removed major labels May 10, 2017
@wooldridge wooldridge modified the milestones: 9.0-3, 9.0-2 May 10, 2017
@wooldridge wooldridge modified the milestones: 9.0-4, 9.0-3 Jul 24, 2017
@wooldridge wooldridge modified the milestones: 2.1.1, 3.0.1 Nov 16, 2017
@georgeajit georgeajit modified the milestones: 3.0.1, 2.4.next Apr 30, 2020
@georgeajit georgeajit assigned ehennum and unassigned wooldridge Apr 30, 2020
@ehennum ehennum modified the milestones: 2.5.0, 2.5-NEXT Nov 16, 2020
@anu3990 anu3990 modified the milestones: 2.5-NEXT, node-client-api-NEXT Jun 9, 2021
@ehennum ehennum added will-not-fix and removed new labels Feb 15, 2022
@ehennum
Copy link
Contributor

ehennum commented Feb 15, 2022

Can be reopened if this becomes an issue.

@ehennum ehennum closed this as completed Feb 15, 2022
@ehennum ehennum removed their assignment Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants