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.

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

Thread commissioning fix #5836

Merged
merged 2 commits into from
Jan 19, 2018
Merged

Conversation

artokin
Copy link
Contributor

@artokin artokin commented Jan 12, 2018

Description

Fix issue PelionIoT/coap-service#88.

Status

READY

Steps to test or reproduce

Device commissioning in Thread network was broken. With this fix the commissioning is working like described in: https://os.mbed.com/docs/v5.7/tutorials/mesh.html#thread-commissioning

Arto Kinnunen added 2 commits January 12, 2018 14:02
… from d0a2597..8689fca

8689fca Reverting address check from transaction find (ARMmbed#89)
ca3c3ab Fix transactions to handle all token lengths (ARMmbed#87)

git-subtree-dir: features/nanostack/FEATURE_NANOSTACK/coap-service
git-subtree-split: 8689fca
* commit '47341478379429c08c468093e0326b5c79ecbc62':
  Squashed 'features/nanostack/FEATURE_NANOSTACK/coap-service/' changes from d0a2597..8689fca
@artokin
Copy link
Contributor Author

artokin commented Jan 12, 2018

@adbridge , @0xc0170 would you please include this PR to the next patch release.

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 12, 2018

/morph build

@adbridge
Copy link
Contributor

@artokin The deadline for 5.7.3 has passed so we will get this into 5.7.4 in 2 weeks time.

@mbed-ci
Copy link

mbed-ci commented Jan 12, 2018

Build : SUCCESS

Build number : 852
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5836/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build

@mbed-ci
Copy link

mbed-ci commented Jan 12, 2018

@mbed-ci
Copy link

mbed-ci commented Jan 12, 2018

@0xc0170
Copy link
Contributor

0xc0170 commented Jan 15, 2018

/morph uvisor-test

@cmonr
Copy link
Contributor

cmonr commented Jan 18, 2018

@artokin, is this PR still needed, considering the original issue has already been resolved?

@artokin
Copy link
Contributor Author

artokin commented Jan 19, 2018

@cmonr , yes this fix is still needed. This will bring coap-service fix to Mbed-OS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants