Skip to content

Conversation

@SamRemis
Copy link
Member

Issue #, if available:

Description of changes:
User agent header updated to include info on OS and language version

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

@SamRemis SamRemis requested a review from ajredniwja June 10, 2021 14:47
@SamRemis SamRemis merged commit 71b79a0 into aws:master Jun 22, 2021
cweiske added a commit to mogic-le/aws-sdk-php that referenced this pull request May 12, 2025
The header blacklist contains lowercase header names only,
except "X-Amz-User-Agent". This causes this header not to be filtered out,
leading to an error with some proxies that remove the empty header:
> There were headers present in the request which were not signed.

This commit lowercases the header, which correctly removes the header
from the signature.

Related: aws#2259
Resolves: aws#3119
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.

2 participants