Skip to content

feat: x-amz-user-agent for browser#826

Merged
srchase merged 2 commits intoaws:smithy-codegenfrom
srchase:browser-user-agent
Jan 31, 2020
Merged

feat: x-amz-user-agent for browser#826
srchase merged 2 commits intoaws:smithy-codegenfrom
srchase:browser-user-agent

Conversation

@srchase
Copy link
Contributor

@srchase srchase commented Jan 31, 2020

For browser runtime, switch to x-amz-user-agent for user-agent header instead of user-agent.

Fixes: #792

Also cleans up unused functions.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aws-sdk-js-automation
Copy link

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@codecov-io
Copy link

codecov-io commented Jan 31, 2020

Codecov Report

❗ No coverage uploaded for pull request base (smithy-codegen@2b64381). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                @@
##             smithy-codegen     #826   +/-   ##
=================================================
  Coverage                  ?   92.36%           
=================================================
  Files                     ?      147           
  Lines                     ?     2907           
  Branches                  ?      508           
=================================================
  Hits                      ?     2685           
  Misses                    ?      222           
  Partials                  ?        0
Impacted Files Coverage Δ
packages/util-user-agent-node/src/index.ts 100% <ø> (ø)
packages/util-user-agent-browser/src/index.ts 83.33% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b64381...ccde224. Read the comment docs.

@aws-sdk-js-automation
Copy link

AWS CodeBuild CI Report

  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@srchase srchase merged commit 6c3e098 into aws:smithy-codegen Jan 31, 2020
@srchase srchase deleted the browser-user-agent branch January 31, 2020 23:40
@lock
Copy link

lock bot commented Feb 8, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators Feb 8, 2020
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.

4 participants