[extension/bearertokenauth] Fix the RPC credential updates #35653
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.
Fixes a bug - The gRPC headers not getting updated after collecter startup
Description
Does not appear there were any open/closed issues for this, but somewhere down the line the extension handling code has likely changed so that the
PerRPCCredentials
does not get called anymore for each call.To excite the issue - you can start the collector with valid credentials, then inject invalid credentials to the token file.
Expected behavior: The collector starts using the invalid credentials and fails authentication
Old behavior: The collector keeps using the credentials it read at startup (problematic if you need to refresh the token)
This fixes the behavior by moving the evaluation of the authorization token to happen when the metadata is being fetched instead of at a time when the auth object is created.
Testing
Manual testing performed by manipulating the token file on the disk.
The interface to the rest of the application should remain exactly the same.