Skip to content

Conversation

@shivaspeaks
Copy link
Member

No description provided.

@shivaspeaks
Copy link
Member Author

I'm working on fixing the unit tests. Until then please take a look and provide suggestions/ improvements.

Copy link
Contributor

@kannanjgithub kannanjgithub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review comments.

@shivaspeaks shivaspeaks marked this pull request as ready for review March 25, 2025 17:50
@shivaspeaks shivaspeaks requested a review from ejona86 March 25, 2025 17:50
@shivaspeaks shivaspeaks requested a review from ejona86 March 26, 2025 15:58
Copy link
Contributor

@kannanjgithub kannanjgithub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review comments.

Copy link
Contributor

@kannanjgithub kannanjgithub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review comments.

Copy link
Contributor

@kannanjgithub kannanjgithub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some minor comments.

Copy link
Contributor

@kannanjgithub kannanjgithub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Forgot the catching runtime exception comment.

@shivaspeaks shivaspeaks merged commit 84c7713 into grpc:master Apr 2, 2025
16 checks passed
@shivaspeaks shivaspeaks deleted the connect_audience_in_gcp_filter branch April 2, 2025 11:00
Sangamesh1997 pushed a commit to Sangamesh1997/grpc-java that referenced this pull request Apr 3, 2025
shivaspeaks added a commit to shivaspeaks/grpc-java that referenced this pull request May 22, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants