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

doc: mgmt: mcumgr: img_mgmt: Documentation specifies that hash in state of images is a required field #54064

Closed
nordicjm opened this issue Jan 24, 2023 · 1 comment · Fixed by #56311
Closed

doc: mgmt: mcumgr: img_mgmt: Documentation specifies that hash in state of images is a required field #54064

nordicjm opened this issue Jan 24, 2023 · 1 comment · Fixed by #56311
Assignees
@nordicjm
Labels
area: mcumgr bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Milestone
v3.4.0

Comments

@nordicjm
Copy link
Collaborator

Describe the bug
On https://docs.zephyrproject.org/latest/services/device_mgmt/smp_groups/smp_group_1.html it states that for the "get image state" response, hash is a required field, however in mcuboot with serial recovery, this field is in fact not sent, therefore either the field should be sent or the documentation needs to be updated to specify that it is not a required field in the response.

This issue has been opened in mcuboot also: mcu-tools/mcuboot#1581
@nordicjm nordicjm added bug The issue is a bug, or the PR is fixing a bug area: mcumgr labels Jan 24, 2023
@nordicjm nordicjm mentioned this issue Jan 24, 2023
Closed
@laurenmurphyx64 laurenmurphyx64 added the priority: low Low impact/importance bug label Jan 24, 2023
@carlescufi carlescufi added this to the v3.4.0 milestone Feb 7, 2023
@nordicjm
Copy link
Collaborator Author

Submitted fix for mcuboot: mcu-tools/mcuboot#1644

@nordicjm nordicjm assigned nordicjm and unassigned de-nordic Mar 28, 2023
@nordicjm nordicjm mentioned this issue Mar 29, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nordicjm nordicjm

Labels
area: mcumgr bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug
Projects
None yet
Milestone
v3.4.0
Development

Successfully merging a pull request may close this issue.

doc: device_mgmt: smp_groups: smp_group_1: Add upload hash note nordicjm/zephyr
4 participants
@carlescufi @laurenmurphyx64 @nordicjm @de-nordic