Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve error handling #7

Open
1 of 3 tasks
jiayihu opened this issue Nov 26, 2020 · 0 comments
Open
1 of 3 tasks

Improve error handling #7

jiayihu opened this issue Nov 26, 2020 · 0 comments
Labels
enhancement New feature or request epic fedra-iot
Projects
Agenda

Comments

@jiayihu
Copy link
Owner

jiayihu commented Nov 26, 2020 •
edited

Currently the code panics on error, it should instead improve error handling collecting the different error struct in a single struct

  • Define a unique error that includes all wasmi errors
  • Minimize usage of .unwrap and .expect
  • Return proper CoAP response codes for errors implemented as "separate response" (RFC 7252 5.2.2).
@jiayihu jiayihu added this to Backlog in Agenda Nov 26, 2020
@jiayihu jiayihu added enhancement New feature or request fedra-iot labels Nov 26, 2020
@jiayihu jiayihu moved this from Backlog to In progress in Agenda Dec 2, 2020
@jiayihu jiayihu moved this from In progress to Backlog in Agenda Dec 2, 2020
@jiayihu jiayihu added the epic label Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request epic fedra-iot
Projects
Agenda
Backlog
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jiayihu