Skip to content

Commit 2337588

Browse files
authored
Fixed issue where HTTP headers set in a ClientOptions object passed to a client builder would not be set on a client instantiated by said builder. (Azure#22462)
* Fixed issue where HTTP headers set in a `ClientOptions` object passed to a client builder would not be set on a client instantiated by said builder. * Updated CHANGELOG.
1 parent b507f4a commit 2337588

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

sdk/tables/azure-data-tables/CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
## 12.1.0-beta.1 (Unreleased)
44

5+
### Bug Fixes
6+
- Fixed issue where HTTP headers set in a `ClientOptions` object passed to a client builder would not be set on a client instantiated by said builder.
57

68
## 12.0.0 (2021-06-11)
79

sdk/tables/azure-data-tables/src/main/java/com/azure/data/tables/BuilderHelper.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,13 @@ static HttpPipeline buildPipeline(
6969
CoreUtils.getApplicationId(clientOptions, logOptions), CLIENT_NAME, CLIENT_VERSION, configuration));
7070
policies.add(new RequestIdPolicy());
7171

72-
List<HttpHeader> httpHeaderList = new ArrayList<>();
73-
7472
if (clientOptions != null) {
73+
List<HttpHeader> httpHeaderList = new ArrayList<>();
74+
7575
clientOptions.getHeaders().forEach(header ->
7676
httpHeaderList.add(new HttpHeader(header.getName(), header.getValue())));
77+
78+
policies.add(new AddHeadersPolicy(new HttpHeaders(httpHeaderList)));
7779
}
7880

7981
// Add per call additional policies.

0 commit comments

Comments
 (0)