-
Notifications
You must be signed in to change notification settings - Fork 9
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
Bug in support for RFC 8525 #60
Comments
i think this area has not received a lot of interop testing, so if you see
something that looks off, it likely is.
…On Wed, Apr 17, 2024 at 11:21 AM bart ***@***.***> wrote:
Hi!
If I interpret the supported
<https://freeconf.org/docs/reference/compliance/rfcs/> RFC 8525
<https://datatracker.ietf.org/doc/html/rfc8525> correctly, the container
modules-state from restconf/data/ietf-yang-library is deprecated in favor
of a container yang-library. However, when I call the endpoint on the Car
example, the reply contains modules-state but not yang-library. Is this a
bug?
—
Reply to this email directly, view it on GitHub
<#60>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAACA7WTKVPG6ZZBSLZ5WILY52HQJAVCNFSM6AAAAABGLRMWCKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2DQNJWGAZTKOI>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Thank you for the swift reply. I find an excuse to work on this (no guarantees), I could make a PR. Would you be open to that? From what I can see, this issue requires extending device/yang_lib_node.go and device/yang_lib_test.go. Would you agree? |
absolutely. Yes, those appear to be the right files.
…On Tue, Apr 30, 2024 at 6:26 AM bart ***@***.***> wrote:
Thank you for the swift reply. I find an excuse to work on this (no
guarantees), I could make a PR. Would you be open to that? From what I can
see, this issue requires extending device/yang_lib_node.go
<https://github.com/freeconf/restconf/blob/master/device/yang_lib_node.go>
and device/yang_lib_test.go
<https://github.com/freeconf/restconf/blob/master/device/yang_lib_test.go>.
Would you agree?
—
Reply to this email directly, view it on GitHub
<#60 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAACA7QCN4HAE6RII2OISY3Y75WWLAVCNFSM6AAAAABGLRMWCKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBUHEZTIMRTGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
This was referenced May 6, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi!
If I interpret the supported RFC 8525 correctly, the container
modules-state
fromrestconf/data/ietf-yang-library
is deprecated in favor of a containeryang-library
. However, when I call the endpoint on the Car example, the reply containsmodules-state
but notyang-library
. Is this a bug?The text was updated successfully, but these errors were encountered: