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

DEVEXP-111: add User-Agent SDK header #21

Merged
merged 3 commits into from
Nov 30, 2023

Conversation

JPPortier
Copy link
Contributor

Sending SDK User-Agent header.
Format will be: sinch-sdk/{sdk_version} ({language}/{language_version}; {implementation_type}; {auxiliary_flag})

Where:

  • sdk_version: Version according to release/deployment version
  • language: hard coded to Java
  • language_version: java compiler version set for sources
  • implementation_type: Currently only Apache for HTTP Apache client is supported
  • auxiliary_flag: Comma separated list containing information about JVM used for SinchClient execution:
    • java.vendor
    • java.version

Example of generated header: User-Agent: sinch-sdk/0.0.1-SNAPSHOT (Java/1.8; Apache; AdoptOpenJDK,14.0.2)

Copy link
Contributor

github-actions bot commented Nov 28, 2023

Dependency Review

✅ No vulnerabilities or license issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA ae5a621.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Manifest Files

pom.xml

@JPPortier JPPortier force-pushed the devexp-111-add-user-agent-sdk-header branch from 49e0535 to 9d655cd Compare November 29, 2023 07:48
@JPPortier JPPortier force-pushed the devexp-111-add-user-agent-sdk-header branch from 0f600bb to ae5a621 Compare November 30, 2023 06:23
@JPPortier JPPortier merged commit b1df78d into main Nov 30, 2023
4 checks passed
@JPPortier JPPortier deleted the devexp-111-add-user-agent-sdk-header branch November 30, 2023 06:28
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