You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/03/08 02:02:35 UTC
[GitHub] [incubator-nuttx-apps] yamt opened a new pull request #1050: webclient: Remove a duplicated "Connection:" header
yamt opened a new pull request #1050:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1050
## Summary
When making the following change, I haven't noticed that
g_httpuseragentfields contains a "Connection" header.
```
commit 092ce8144445a9b287554e59e3150bab3d67bedb
Author: YAMAMOTO Takashi <ya...@midokura.com>
Date: Mon Mar 7 09:30:23 2022 +0900
webclient: Always use "connection: close" for HTTP 1.1 for now
* This matches the HTTP 1.0 behavior.
* Persistent connection doesn't make much sense with the current API.
```
It seems that some servers are not happy with the duplicated header
and ignore them. (and do the default keep-alive for HTTP 1.1)
eg. Azure Blob (global)
## Impact
## Testing
tested against Azure Blob.
--
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: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #1050: webclient: Remove a duplicated "Connection:" header
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #1050:
URL: https://github.com/apache/incubator-nuttx-apps/pull/1050
--
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: commits-unsubscribe@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org