Skip to content

Conversation

abnegate
Copy link
Member

@abnegate abnegate commented Jul 18, 2025

What does this PR do?

  • Update sdk to use swift-native doc comments instead of jsdoc styled comments as per Swift Documentation Comments
  • Add incrementDocumentAttribute and decrementDocumentAttribute support to Databases service
  • Add gif support to ImageFormat enum
  • Remove Content-Type, Content-Length headers and body from websocket requests

Test Plan

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

yes.

@@ -48,8 +48,6 @@ extension HTTPHandler : ChannelInboundHandler, RemovableChannelHandler {
var headers = HTTPHeaders()

headers.add(name: "Host", value: "\(client.host):\(client.port)")
headers.add(name: "Content-Type", value: "text/plain")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this intended and no longer required?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Intended, this broke with proxies because some don't like content-type and content-length or a body when making a GET request

@loks0n loks0n merged commit 6de86ee into main Jul 26, 2025
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.

4 participants