-
Notifications
You must be signed in to change notification settings - Fork 185
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
client: allow UserAgent to be set #119
Conversation
Hello, rebuild the feature as a |
4a79164
to
70e3704
Compare
Codecov Report
@@ Coverage Diff @@
## main #119 +/- ##
==========================================
+ Coverage 84.24% 84.25% +0.01%
==========================================
Files 56 56
Lines 5724 5729 +5
==========================================
+ Hits 4822 4827 +5
Misses 636 636
Partials 266 266
Continue to review full report at Codecov.
|
70e3704
to
78a7d08
Compare
78a7d08
to
76d766e
Compare
Done. |
merged, thanks |
This issue is being locked automatically because it has been closed for more than 6 months. |
It's helpful for debugging to be able to override the hardcoded
gortsplib
User-Agent header, as we can have multiple gortsplib-based clients. I'd also be open to making this a config option or a parameter to the Client, but this was the lowest impact change.It also has the advantage that this parameter can be changed by e.g., docker image without rebuilding.