drill
DRILL-8155: Introduce New Plugin Authentication Modes
#2516

Merged

Overview Commits 33 Changes View On GitHub

DRILL-8155: Introduce New Plugin Authentication Modes #2516

cgivre merged 33 commits into apache:master from jnturton:jdbc-auth-modes

cgivre assigned

cgivre 2 years ago

cgivre force pushed from c0214cd6 to 38d5aea9 2 years ago

cgivre changed the title ~~[WIP] DRILL-8155: Add Impersonation Support for Non-Hadoop Based Storage Plugins~~ DRILL-8155: Add Impersonation Support for Non-Hadoop Based Storage Plugins 2 years ago

cgivre requested a review from

jnturton 2 years ago

cgivre requested a review from

vvysotskyi 2 years ago

cgivre added enhancement

cgivre added doc-impacting

cgivre added security

cgivre added access-controls

jnturton changed the title ~~DRILL-8155: Add Impersonation Support for Non-Hadoop Based Storage Plugins~~ DRILL-8155: Introduce new plugin authentication modes 2 years ago

cgivre changed the title ~~DRILL-8155: Introduce new plugin authentication modes~~ DRILL-8155: Introduce New Plugin Authentication Modes 2 years ago

jnturton force pushed from cd1481b7 to b4448b86 2 years ago

jnturton force pushed from b4448b86 to ce99272d 2 years ago

jnturton force pushed from ce99272d to 44b41581 2 years ago

jnturton force pushed from 44b41581 to 5f926cdc 2 years ago

jnturton force pushed from 5f926cdc to 4c0bfcc0 2 years ago

jnturton force pushed from 4c0bfcc0 to 7277123b 2 years ago

jnturton force pushed from 7277123b to 82c9418b 2 years ago

jnturton force pushed from ecb96d68 to 436abcf8 2 years ago

jnturton force pushed from 436abcf8 to ceba8ef6 2 years ago

jnturton force pushed from ceba8ef6 to fa82282b 2 years ago

jnturton force pushed from fa82282b to cacbbdc9 2 years ago

jnturton force pushed from cacbbdc9 to 6fb4ba73 2 years ago

jnturton force pushed from 6fb4ba73 to eeed51d6 2 years ago

jnturton force pushed from eeed51d6 to ff14630c 2 years ago

cgivre marked this pull request as ready for review 2 years ago

cgivre force pushed from 6ae5f29c to 42668d10 2 years ago

vvysotskyi commented on 2022-04-21

Do not set the read-only hint on JDBC connections.

b78c90e6

Outline of different auth modes in storage-jdbc.

b898dfd9

Rebased to current master and build fixes

89c2daa4

Build works, cred stuff added

c7a52af8

Credentials being saved and pushed down to storage plugin

6f9d03b9

UI now closing properly

533c4f54

User Translation working for HTTP plugin

702151de

HTTP unit tests passing

71da47a9

WIP

adb68123

Fixed import

a3433651

User Credentials now being stored in credential provider

d1af31ad

Working

a5df9ed3

Fixed TPCH Unit Tests

f5b73223

Fix CredProvider SerDe Test

0fefc4b2

Added unit tests for JDBC

ebcaff2a

Code cleanup

4e28e9fa

Fix LGTM alerts

11a0a8e8

Correct username now populating Group Scan

756d3e31

Username to Subscan

f68fb992

Remove PerUserUsernamePasswordCredentials class.

f674b55c

Remove getUserCredentials from CredentialsProvider.

be944e49

Planning errors fixed

ecdab487

Removed unused imports

48fabdca

Fixed minor issues

b19fb8ea

Unit test fixes

66f72275

WIP.

fe4d4e08

Fix CodeQL Alert

e5c277ac

Ignore LGTM False Positive

0d159c7c

Fix tainted string LGTM alert

8282309e

Revert LGTM Comment

14339a47

Addressed review comments

1b2de797

Use fixed size Guava caches in JDBC convetion and dialect factories.

1ca7ff52

jnturton force pushed from c72ba9af to 1ca7ff52 2 years ago

Add a TTL to the JDBC dialect and convention caches.

a99c4349

vvysotskyi approved these changes on 2022-04-26

cgivre merged 7c373232 into master 2 years ago

Reviewers

vvysotskyi

jnturton

Assignees

cgivre

Labels

enhancement doc-impacting security access-controls

Milestone

No milestone

drill DRILL-8155: Introduce New Plugin Authentication Modes #2516 Merged

DRILL-8155: Introduce New Plugin Authentication Modes #2516

drill
DRILL-8155: Introduce New Plugin Authentication Modes
#2516

Merged