Re: [web-bluetooth] Missing API for GATT Write w/o Response (#238)

I would leave the "best effort write" exactly as is - whatever that is. It
should only remain to ensure backward compatibility, and it should
potentially come with a note to not use it in new projects - perhaps even
be removed from the documentation entirely (it should have never existed in
the first place).Changing the method will break implementations. I should
note that in one of my implementations the GATT server device offered both
WriteNoResponse and WriteWithResponse for a characteristic used for
firmware upgrade over the air. Unfortunaltely writeValue() in my case
picked WriteWithResponse and the firmware upgrade took about twice as long.
Changing the GATT options to only offer WriteNoResponse fixed this issue,
but I could only do that because I am in control of the device firmware
(probably unlike most people).

On Wed, Apr 29, 2020 at 10:45 AM Ovidio Ruiz-Henríquez <
notifications@github.com> wrote:

> Please correct me if I'm wrong, but it sounds like there's still agreement
> to have two explicit methods for write with/without response while keeping
> the original writeValue. This change was already added to the spec with
> #433 <https://github.com/WebBluetoothCG/web-bluetooth/pull/433>. It also
> looks like there's agreement that the specified write should be done
> regardless of the characteristic properties. I also agree with these ideas.
>
> If that's the case, what's left is to agree on a default algorithm for
> writeValue(), which @scheib <https://github.com/scheib> and @g-ortuno
> <https://github.com/g-ortuno> prefer writeWithResponse(). Would this
> algorithm also be the "best effort write" @MikeTheDane
> <https://github.com/MikeTheDane> and @bluetooth-mdw
> <https://github.com/bluetooth-mdw> ?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/WebBluetoothCG/web-bluetooth/issues/238#issuecomment-621362590>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ACNNFKL7SZOWLKIQVQQJB7TRPBRTHANCNFSM4CC422ZQ>
> .
>


-- 
GitHub Notification of comment by MikeTheDane
Please view or discuss this issue at https://github.com/WebBluetoothCG/web-bluetooth/issues/238#issuecomment-621376548 using your GitHub account

Received on Wednesday, 29 April 2020 18:12:38 UTC