Skip to content

rust, python: Pass client ID for service callback in Request #244

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

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

gasmith
Copy link
Contributor

@gasmith gasmith commented Feb 25, 2025

From discussion on #239: Let's not pass around a separate Client to service callbacks. Instead, make client metadata accessible on the Request.

@gasmith gasmith self-assigned this Feb 25, 2025
Copy link

linear bot commented Feb 25, 2025

@gasmith gasmith force-pushed the gasmith/fg-10614-svc-client branch from a6847fe to d075457 Compare February 25, 2025 23:32
@gasmith gasmith requested review from eloff and bryfox February 25, 2025 23:33
@gasmith gasmith marked this pull request as ready for review February 25, 2025 23:33
Copy link
Contributor

@bryfox bryfox left a comment

Choose a reason for hiding this comment

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

Looks good to me

@gasmith gasmith merged commit 5df1331 into main Feb 26, 2025
29 checks passed
@gasmith gasmith deleted the gasmith/fg-10614-svc-client branch February 26, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants