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
Refactor configauth, getters use the map instead of iteration #4234
Refactor configauth, getters use the map instead of iteration #4234
Conversation
Signed-off-by: Bogdan Drutu <bogdandrutu@gmail.com>
0ec18d7
to
90f5c8a
Compare
Codecov Report
@@ Coverage Diff @@
## main #4234 +/- ##
==========================================
- Coverage 87.93% 87.92% -0.02%
==========================================
Files 174 173 -1
Lines 10265 10263 -2
==========================================
- Hits 9027 9024 -3
- Misses 992 993 +1
Partials 246 246
Continue to review full report at Codecov.
|
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.
Change looks good, just a non-blocking question for more context.
for id, ext := range extensions { | ||
// GetServerAuthenticator attempts to select the appropriate ServerAuthenticator from the list of extensions, | ||
// based on the requested extension name. If an authenticator is not found, an error is returned. | ||
func (a Authentication) GetServerAuthenticator(extensions map[config.ComponentID]component.Extension) (ServerAuthenticator, error) { |
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.
Will this eventually be split into a GetHTTPServerAuthenticator and a GetGRPCServerAuthentiator?
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 think so, I had exactly the same question in my mind while doing the refactoring. @jpkrohling for you to see this comment.
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 just created a new issue, to keep track of it.
Signed-off-by: Bogdan Drutu bogdandrutu@gmail.com