This repository was archived by the owner on Jan 16, 2026. It is now read-only.

feat: authentication using CSP#148

Merged

LukeWinikates merged 1 commit intomasterfrom

K8SSAAS-2215-csp

Aug 30, 2023

Contributor

LukeWinikates commented Aug 18, 2023

No description provided.

LukeWinikates commented

View reviewed changes

internal/reporter.go Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

internal/token/token_csp.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

internal/token/token_csp.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

internal/token/token_csp.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

internal/token/token_csp.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/client_factory.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/client_factory.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/client_factory.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/client_factory.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/client_factory.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/client_factory.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/client_test.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

test/helpers.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

senders/example_newsender_csp_test.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

Contributor Author

LukeWinikates left a comment

added several comments

LukeWinikates commented

View reviewed changes

internal/token/token_csp.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

internal/token/token_csp.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

internal/token/token_csp_test.go Outdated Show resolved Hide resolved

LukeWinikates commented

View reviewed changes

internal/token/token_csp_test.go Outdated Show resolved Hide resolved

LukeWinikates force-pushed the K8SSAAS-2215-csp branch from 7653238 to af85e91 Compare

August 23, 2023 21:23

LukeWinikates requested review from jbooherl, priyaselvaganesan and suprajanarasimhan

August 23, 2023 21:24

LukeWinikates force-pushed the K8SSAAS-2215-csp branch from af85e91 to 7ffed3b Compare

August 23, 2023 21:39

LukeWinikates mentioned this pull request

rename files in senders package to match their contents #147

Closed

LukeWinikates force-pushed the K8SSAAS-2215-csp branch 2 times, most recently from 6f39406 to d88633b Compare

August 24, 2023 15:58

Contributor Author

LukeWinikates commented Aug 28, 2023 •

edited

Loading

needs orgID option
~~support ALL PERMISSIONS scope~~ we dropped the SDK-side scope check instead, because it's redundant given that the server will be doing the checks.

LukeWinikates force-pushed the K8SSAAS-2215-csp branch 2 times, most recently from afd31b3 to 1f9f983 Compare

August 30, 2023 18:06


          feat: authentication using CSP

bdd9b79

Signed-off-by: Ernst Riemer <eriemer@vmware.com>
Signed-off-by: Luke Winikates <winikatesl@vmware.com>

LukeWinikates force-pushed the K8SSAAS-2215-csp branch from 1f9f983 to bdd9b79 Compare

August 30, 2023 18:30

priyaselvaganesan approved these changes

View reviewed changes

LukeWinikates merged commit 4aa3472 into master

LukeWinikates deleted the K8SSAAS-2215-csp branch

October 11, 2023 19:40

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet