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

[Evgeny Grin]

Hi Alexander,

Applied as suggested. Also modified other similar examples with wrong 
API usage.
Thanks for spotting this.

--
All the best,
Evgeny

On 15.06.2022 13:45, Irion, Alexander wrote:
> Please, see the attached small fix:
>
> Remove double adding of MHD_HTTP_HEADER_CONNECTION response header
>
> MHD_create_response_for_upgrade() already adds the 
> MHD_HTTP_HEADER_CONNECTION response header, so
>
> the additional MHD_add_response_header is wrong.
>
> 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_irion@mentor.com 
> <mailto:alexander_irion@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