Closed nigelmegitt closed 2 years ago
@pes10k this has reached our 14 day review minimum period, so we'd like to progress it, and would really appreciate your review feedback to confirm if it addresses the issue (#29), when you're able to take a look.
We have this on the agenda for the PING chairs call on Monday. I will follow up with you all on Monday after that call
On Mar 4, 2022, at 05:51, Nigel Megitt @.***> wrote:
@pes10k this has reached our 14 day review minimum period, so we'd like to progress it, and would really appreciate your review feedback to confirm if it addresses the issue (#29), when you're able to take a look.
— Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you were mentioned.
I'm helping to review with @pes10k and expect to have feedback by Wednesday (9 March 2022).
Commented on issue #29. This PR doesn't specify error handling behavior in much detail, which could have implications for applications that use it, but I suspect privacy and security impacts are very limited.
@pes10k @npdoty thank you, looking at these comments and the comment on #29, my understanding is:
On that basis I will merge this; if there are additional changes needed I would ask that they are opened as new issues.
Appendix is informative and describes Error Reporting and Exception Handling.
Closes #29.
Preview | Diff