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

Allow ByteString Variant to be None #1539

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

ldet
Copy link
Contributor

@ldet ldet commented Dec 22, 2023

Hello,
while working with an open62541 based server, I encountered an empty/null ByteString. The ua_binary implementation of ByteString (_Bytes) already handles this case as per the standard (Part 6 5.2.2.7) and sets the value to None, but the implementation of Variant currently throws an UaError when a ByteString with value None is encountered. This PR adds ByteString to the allowed VariantTypes that can be None.

@oroulet oroulet merged commit b9e2af4 into FreeOpcUa:master Dec 22, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants