Skip to content
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

Add User-Agent with the App Data #1192

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

JcMinarro
Copy link
Member

🎯 Goal

Include a custom User-Agent to any HTTP Call our SDK does. The User-Agent will looks like:

"User-Agent:Stream Video Calls (Development) / 1.1.7-DEBUG(1); StandAloneInstall; (Google; sdk_gphone64_x86_64; SDK 32; Android 12)"

The same we did on GetStream/stream-chat-android#5415

@aleksandar-apostolov aleksandar-apostolov enabled auto-merge (squash) September 27, 2024 14:08
Copy link
Member

@skydoves skydoves 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!

@JcMinarro JcMinarro disabled auto-merge October 14, 2024 13:31
@JcMinarro JcMinarro marked this pull request as draft October 14, 2024 13:31
@JcMinarro
Copy link
Member Author

Do not merge it yet, a customer is having an issue with the same implementation on Chat SDK when the app name contains strange characters

@JcMinarro JcMinarro force-pushed the feature/use-app-user-agent branch from 09616b9 to 8df6373 Compare October 15, 2024 12:32
@JcMinarro
Copy link
Member Author

Do not merge it yet, a customer is having an issue with the same implementation on Chat SDK when the app name contains strange characters

Ok, it is fixed on the last commit

@JcMinarro JcMinarro marked this pull request as ready for review October 15, 2024 12:33
@JcMinarro JcMinarro force-pushed the feature/use-app-user-agent branch 2 times, most recently from f0787f5 to c2a729d Compare October 15, 2024 12:46
@JcMinarro JcMinarro force-pushed the feature/use-app-user-agent branch from c2a729d to 94242b4 Compare October 15, 2024 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants