-
Notifications
You must be signed in to change notification settings - Fork 789
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
TINKERPOP-1566 Kerberos authentication for gremlin-server #534
Merged
Merged
Changes from all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
b3cb7e9
Kerberos authenticator files added
36713cf
Added failing test shouldAuthenticateWithSerializeResultToString
d2639a7
Fixed problem with non-lowercase hostname
2ead423
Added Kerberos tests for client
202d658
Added Krb5Authenticator to reference docs
4544b01
Made immutable variables final
14dd521
Refactored and cleaned directory-kerby usage
972b558
Verified and documented proper usage of sasl configuration properties
cb75d9b
Removed unnecessary test dependency
68eff1d
Added audit logging wih tests and docs
5c735d8
KdcFixture closing
c364643
Fixed issue for byte[] when using serializeResultToString
c849295
Changelog due to TINKERPOP-1566
6a9c667
Some cleanup
a5f3e1f
Improved code comments regarding authorization after Kerberos authent…
0ca1b16
Added audit log integrate test for two clients
d81f0d7
Adapted to TINKERPOP-1600 after rebase
3668557
Added Kerberos authentication test with ssl
6400128
Restored WARN for gremlin-server log4j.properties
b878318
Fixed failing gremlin-server authentication integration tests
0e299b0
Deleted unnecessary Kerberos config file
4e40df9
Restored gremlin-driver behavior regarding GSSAPI mechanism
1bbb0b2
Merge branch 'master' into TINKERPOP-1566
91e510c
Fixed use of socket address for audit logging
3c321ca
Merge branch 'TINKERPOP-1566' of https://github.com/vtslab/incubator-…
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be in alphabetical order?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Answer to this riddle (required some thought on my part): also other config items, like scriptEngines.<name>.config, are at the bottom of their section. Just because it looks more orderly in the config file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What I meant is that this one was not in alphabetical order and perhaps it should be.