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

Bind response - 0x8030 #122

Open
pipiche38 opened this issue Jan 16, 2019 · 8 comments

Comments

@pipiche38
Copy link

commented Jan 16, 2019

No description provided.

@pipiche38

This comment has been minimized.

Copy link
Author

commented Jan 16, 2019

Would it be possible to have the source address in the bind response ?
either the IEEE or Short address would be welcome

@ISO-B

This comment has been minimized.

Copy link
Collaborator

commented Jan 17, 2019

Adding following lines

ZNC_BUF_U8_UPD ( &au8LinkTxBuffer [u16Length], psStackEvent->uEvent.sApsDataIndEvent.u8SrcEndpoint, u16Length );
ZNC_BUF_U8_UPD ( &au8LinkTxBuffer [u16Length], psStackEvent->uEvent.sApsDataIndEvent.u8SrcAddrMode, u16Length );

after

ZNC_BUF_U8_UPD ( &au8LinkTxBuffer [u16Length] , sApsZdpEvent.uZdpData.sUnbindRsp.u8Status, u16Length );

should do the job. I can't see any reason why this couldn't be added.

@pipiche38

This comment has been minimized.

Copy link
Author

commented Jan 17, 2019

@ISO-B Are you in keybase.io ? Would like to chat in MP

@pipiche38

This comment has been minimized.

Copy link
Author

commented Feb 13, 2019

@ISO-B , @fairecasoimeme
it seems that it has not been implemented in 3.0f , correct ?

@pipiche38 pipiche38 changed the title Bind response - 0x80 Bind response - 0x8030 Feb 13, 2019

@pipiche38

This comment has been minimized.

Copy link
Author

commented Mar 6, 2019

@ISO-B if you can make this one in 3.10, this would be great!

@ISO-B ISO-B self-assigned this Mar 6, 2019

@ISO-B ISO-B added the enhancement label Mar 6, 2019

@ISO-B ISO-B added this to the v3.1a milestone Mar 7, 2019

@ISO-B

This comment has been minimized.

Copy link
Collaborator

commented Mar 7, 2019

This will be in next release version

@ISO-B ISO-B closed this Mar 7, 2019

@pipiche38

This comment has been minimized.

Copy link
Author

commented Mar 7, 2019

Just check your PR. Is there a way to get also the Cluster on which the bind is made ?

@fairecasoimeme fairecasoimeme referenced this issue Jul 12, 2019

Merged

V3.1a #209

@doudz

This comment has been minimized.

Copy link
Contributor

commented Jul 18, 2019

Doc not up to date

@ISO-B ISO-B reopened this Jul 18, 2019

@ISO-B ISO-B removed their assignment Jul 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.