Re: [libmicrohttpd] Fix: Remove double adding of MHD_HTTP_HEADER_CONNECTION response header

[Evgeny Grin]

Hi Alexander,


Thanks for spotting this.

--
All the best,
Evgeny

On 15.06.2022 13:45, Irion, Alexander wrote:

Applied as suggested. Also modified other similar examples with wrong API usage.

Please, see the attached small fix:

Remove double adding of MHD_HTTP_HEADER_CONNECTION response header



the additional MHD_add_response_header is wrong.

MHD_create_response_for_upgrade() already adds the MHD_HTTP_HEADER_CONNECTION response header, so In our application it caused, that on a Apple iPad, the websocket was immediately closed by the browser(Safari, Chrome, Opera) after the protocol upgrade. Browsers on Linux, Android did not had this issue. Signed-off-by: Alexander Irion alexander_ir...@mentor.com <mailto:alexander_ir...@mentor.com>

-----------------

Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

OpenPGP_signature
Description: OpenPGP digital signature