-
Notifications
You must be signed in to change notification settings - Fork 15
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
adds authenticator layer to the SDK for the client and management api #377
Conversation
andrewpmartinez
commented
Apr 18, 2023
- remove conf sub modules, renames config.New* to ziti.NewConfig* removes conflicts with app that have their own conf module
- adds authenciation mechanisms for client/management APIs
- uses new authentication mechanisms for all auth paths, patches old identity.ID configuration for backwards compat
management/authenticators.go
Outdated
@@ -0,0 +1,329 @@ | |||
package ziti |
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.
these functions will be some of the first ones people explore, i'd really appreciate godoc on them once we agree on the api
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.
Sure sure.
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.
I'm going to pause my review and ask for changes i've already commented on. Maybe we can talk in person? I don't think we should merge this as is.
49026a2
to
66fe2aa
Compare
8e4e752
to
ca4556a
Compare
- remove conf sub modules, renames config.New* to ziti.NewConfig* removes conflicts with app that have their own conf module - adds authenciation mechanisms for client/management APIs - uses new authentication mechanisms for all auth paths, patches old identity.ID configuration for backwards compat - remove go.work/sum file, replaced with replace stanzas in example go.mod files - adds doc - de-dupes IdentityProvider interface - renames ApiAuthenticator to Authenticator - moves client API utilities to client_api - moves management API utilities to management_api - adds edge_apis packaged - single authenticator implementation - makes CaPool on client the default unless set on Credentials
ca4556a
to
341b7d0
Compare