Title: [BUG] deferred_response never sends content string passed to constructor · Issue #331 · etr/libhttpserver · GitHub
Open Graph Title: [BUG] deferred_response never sends content string passed to constructor · Issue #331 · etr/libhttpserver
X Title: [BUG] deferred_response never sends content string passed to constructor · Issue #331 · etr/libhttpserver
Description: Prerequisites Put an X between the brackets on this line if you have checked that your issue isn't already filed: https://github.com/search?l=&q=repo:etr/libhttpserver&type=Issues Description The deferred_response constructor include...
Open Graph Description: Prerequisites Put an X between the brackets on this line if you have checked that your issue isn't already filed: https://github.com/search?l=&q=repo:etr/libhttpserver&type=Issues Description T...
X Description: Prerequisites Put an X between the brackets on this line if you have checked that your issue isn't already filed: https://github.com/search?l=&q=repo:etr/libhttpserver&type=Issues D...
Opengraph URL: https://github.com/etr/libhttpserver/issues/331
X: @github
Domain: patch-diff.githubusercontent.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"[BUG] deferred_response never sends content string passed to constructor","articleBody":"### Prerequisites\r\n\r\n* [X] Put an X between the brackets on this line if you have checked that your issue isn't already filed: https://github.com/search?l=\u0026q=repo%3Aetr%2Flibhttpserver\u0026type=Issues\r\n\r\n### Description\r\n\r\nThe `deferred_response` constructor includes a `const std::string\u0026 content = \"\"` parameter that is documented as:\r\n\u003e Besides the callback, optionally, you can provide a content parameter that sets the initial message sent immediately to the client.\r\n\r\nThe `content` parameter is passed to the base class `string_response` constructor where it is assigned to the `content` member but never actually consumed because `string_response::get_raw_response()` is never called.\r\n\r\nThis behavior might appear to have been removed by https://github.com/etr/libhttpserver/commit/ed0841d177d88dada6e21e4dd5c36f8b670c1ee0 in #145, but it likely didn't work even in https://github.com/etr/libhttpserver/commit/eea93bdf45c42ffbb487f06385ab6a7647bc0a5f because `get_raw_response()` is a virtual function.\r\n\r\nThe tests don't expect the content parameter to be sent either: https://github.com/etr/libhttpserver/blob/d249ba682441dbb979146482aff01a7073ed165a/test/integ/deferred.cpp#L164\r\n\r\nAt this point in the interest of bug compatibility maybe just change the parameter name from \"content\" to \"unused\" and fix the docs.\r\n\r\n### Steps to Reproduce\r\n\r\nSend a `deferred_response` with a non-empty `content` argument such as https://github.com/etr/libhttpserver/blob/d249ba682441dbb979146482aff01a7073ed165a/test/integ/deferred.cpp#L103\r\n\r\n**Expected behavior:** Content argument is transmitted.\r\n\r\n**Actual behavior:** Content argument is ignored.\r\n\r\n**Reproduces how often:** 100%\r\n\r\n### Versions\r\n\r\n* OS version `Linux hydrol2 4.18.0-477.15.1.el8_8.x86_64 #1 SMP Fri Jun 2 08:27:19 EDT 2023 x86_64 x86_64 x86_64 GNU/Linux`\r\n* libhttpserver verion 0.18.2 compiled\r\n* libmicrohttpd version 0.9.71 compiled\r\n \r\n### Additional Information\r\n\r\n\r\n","author":{"url":"https://github.com/jcphill","@type":"Person","name":"jcphill"},"datePublished":"2023-08-17T20:43:06.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/331/libhttpserver/issues/331"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:7fc0605e-e326-ea8c-2591-ceebc2e40873 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | E814:376227:16A1FB1:1E9E3E9:69701198 |
| html-safe-nonce | 3bd5d84709a0016ecd65733c6c418852a5855c959d927082016556c84125d88e |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFODE0OjM3NjIyNzoxNkExRkIxOjFFOUUzRTk6Njk3MDExOTgiLCJ2aXNpdG9yX2lkIjoiNzM5OTY4MDcyNjYxOTEzMjMxMiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 686216469dd1d1f682e010b73cf6edd9a8e5c378a9744f65f1b81afee86f9c05 |
| hovercard-subject-tag | issue:1855679984 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/etr/libhttpserver/331/issue_layout |
| twitter:image | https://opengraph.githubassets.com/175f2721988fe58af26b5b47aef898932f14bad5ce4268b98fb98eff70f5533b/etr/libhttpserver/issues/331 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/175f2721988fe58af26b5b47aef898932f14bad5ce4268b98fb98eff70f5533b/etr/libhttpserver/issues/331 |
| og:image:alt | Prerequisites Put an X between the brackets on this line if you have checked that your issue isn't already filed: https://github.com/search?l=&q=repo:etr/libhttpserver&type=Issues Description T... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | jcphill |
| hostname | github.com |
| expected-hostname | github.com |
| None | 043a09521e81f00c45482046b3f38390b8b292cf9a86ad665a632633a072c05c |
| turbo-cache-control | no-preview |
| go-import | github.com/etr/libhttpserver git https://github.com/etr/libhttpserver.git |
| octolytics-dimension-user_id | 1201536 |
| octolytics-dimension-user_login | etr |
| octolytics-dimension-repository_id | 3416891 |
| octolytics-dimension-repository_nwo | etr/libhttpserver |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 3416891 |
| octolytics-dimension-repository_network_root_nwo | etr/libhttpserver |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 98881eadf8ae73000970c42161c094b0809b1330 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width