Skip to content

Authentication on the client side#14

Merged
JakaMohorko merged 14 commits intomainfrom
authentication
Mar 31, 2026
Merged

Authentication on the client side#14
JakaMohorko merged 14 commits intomainfrom
authentication

Conversation

@viacheslauK
Copy link
Copy Markdown
Contributor

@viacheslauK viacheslauK commented Mar 18, 2026

Brief:

This pull request introduces changes to the client-side of the OPC UA module in accordance with the Authentication Modelling Guide. When a user does not have sufficient permissions to read, write, or execute, the corresponding properties and attributes of openDAQ entities are set accordingly. Additionally, the behavior of certain methods is adjusted based on the user’s permissions.

Description

  • Function and Procedure calls return proper error code on client side if user access denied;
  • readOnly flag for properties on client side if there is no write permission;
  • set a method property as invisible on client side if there is no execute permission;
  • BeginUpdate and EndUpdate methods return proper error code on client side;
  • object attributes locking on client side if there is no write access;
  • getAvailableFunctionBlockTypes method returns an empty list on client side if there is no write permission (only for a device);
  • setOperationMode and setOperationModeRecursive methods return proper error code on client side;
  • acceptsSignal method returns false on client side if there is no write permission;
  • tests;

Known issues

  • getAvailableFunctionBlockTypes method has not been implemented for FunctionBlock;
  • getAvailableDeviceTypes and getAvailableDevices methods have not been implemented for Device;
  • lock and 'unlock' methods have not been implemented for Device;

@viacheslauK viacheslauK changed the title Authentication Authentication on the client side Mar 18, 2026
@viacheslauK viacheslauK self-assigned this Mar 18, 2026
@viacheslauK viacheslauK marked this pull request as ready for review March 20, 2026 17:17
Copy link
Copy Markdown
Contributor

@denise-opendaq denise-opendaq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me. just need small changes :)

Copy link
Copy Markdown
Contributor

@denise-opendaq denise-opendaq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

super puper :)

@JakaMohorko JakaMohorko merged commit 1e50636 into main Mar 31, 2026
8 checks passed
@JakaMohorko JakaMohorko deleted the authentication branch March 31, 2026 05:09
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.

3 participants