Skip to content

Commit

Permalink
feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
aajtodd committed Oct 19, 2023
1 parent f107058 commit fb94a1b
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .changes/b1a1bee4-e3e4-4520-ae8e-48d3b0e94ae9.json
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"id": "b1a1bee4-e3e4-4520-ae8e-48d3b0e94ae9",
"type": "bugfix",
"description": "Do not log intermediate signature calculations without explicit opt-in via `LogMode.LogSignging`."
"description": "Do not log intermediate signature calculations without explicit opt-in via `LogMode.LogSigning`."
}
Original file line number Diff line number Diff line change
Expand Up @@ -166,9 +166,9 @@ public class AwsSigningConfig(builder: Builder) {

/**
* Flag enabling whether detailed trace logging is enabled (if the signer implementation supports it). When true
* signers should emit intermediate logging details such as the canonical request and string to sign at the trace
* level. This is an opt-in configuration because these intermediate outputs may contain sensitive fields bound
* to headers, URI, or query parmaeters.
* signers should emit intermediate logging details such as the canonical request at the trace level.
* This is an opt-in configuration because these intermediate outputs may contain sensitive fields bound
* to headers, URI, or query parameters.
*/
public val enableTraceLogging: Boolean = builder.enableTraceLogging

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,8 +45,8 @@ class LogModeTest {
@Test
fun testFromStringComposite() {
assertEquals(
LogMode.fromString("LogRequest|LogRequestWithBody|LogResponse"),
(LogMode.LogRequest + LogMode.LogRequestWithBody + LogMode.LogResponse),
LogMode.fromString("LogRequest|LogRequestWithBody|LogResponse|LogSigning"),
(LogMode.LogRequest + LogMode.LogRequestWithBody + LogMode.LogResponse + LogMode.LogSigning),
)
}

Expand Down

0 comments on commit fb94a1b

Please sign in to comment.