mdedetrich commented on code in PR #1246:
URL: https://github.com/apache/pekko-connectors/pull/1246#discussion_r2459953566


##########
s3/src/main/scala/org/apache/pekko/stream/connectors/s3/impl/S3Stream.scala:
##########
@@ -832,7 +829,6 @@ import scala.util.{ Failure, Success, Try }
       bucket: String,
       method: HttpMethod,
       httpRequest: (HttpMethod, S3Settings) => HttpRequest,
-      headers: Seq[HttpHeader],

Review Comment:
   Yes I noticed this as well, if you want to fix this I would prefer to put it 
in a separate PR so we don't mix this together (its easier to bisect/revert 
changes if they are in their own PR's/commits)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to