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

message-id present on netconf app "hello" #369

Closed
pheller opened this issue Sep 26, 2022 · 2 comments
Closed

message-id present on netconf app "hello" #369

pheller opened this issue Sep 26, 2022 · 2 comments
Labels
bug fixed please verify

Comments

@pheller
Copy link
Contributor

pheller commented Sep 26, 2022 •
edited
Loading

Per RFC 6241 Appendix B, XML Schema, the message-id attribute is legal on rpc and rpc-reply elements, but it is present on the hello element sent by clixon_netconf application.

Strictly validating clients may refuse to communicate with Clixon when this attribute is present.
@pheller pheller changed the title message-id present on netconf backend "hello" message-id present on netconf app "hello" Sep 27, 2022
olofhagsand added a commit that referenced this issue Sep 28, 2022
@olofhagsand
Fixed: [message-id present on netconf app "hello"](#369)
e3f3d77
@olofhagsand
Copy link
Member

Fixed: please verify

@pheller
Copy link
Contributor Author

pheller commented Sep 29, 2022

Works as intended, thanks!

@pheller pheller closed this as completed Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug fixed please verify
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pheller @olofhagsand