Skip to content

sap_hdbsql: Add error handling and secure flags for hdbsql#75

Merged
marcelmamula merged 2 commits into
sap-linuxlab:devfrom
marcelmamula:hdbsql
Feb 27, 2026
Merged

sap_hdbsql: Add error handling and secure flags for hdbsql#75
marcelmamula merged 2 commits into
sap-linuxlab:devfrom
marcelmamula:hdbsql

Conversation

@marcelmamula

Copy link
Copy Markdown
Contributor

Changes

These changes were wrapped in light refactoring for clarity.

Tests

Tested on SLES_SAP 16.0 running BW4HANA database.

Authentication error

        failed: true
        msg: SAP HANA Authentication Failed. Please check credentials/userstore.
        rc: 3
        stderr: |-
            * 10: authentication failed SQLSTATE: 28000

Authorization error:

        failed: true
        msg: 'SAP HANA Authorization Error: The user has insufficient privileges to perform
            this action.'
        rc: 3
        stderr: |-
            * 258: insufficient privilege: Detailed info for this error can be found with guid '250306B97F15FC499AB54C649BEF6B63' SQLSTATE: HY000

@marcelmamula marcelmamula self-assigned this Feb 26, 2026
@marcelmamula marcelmamula added the enhancement New feature or request label Feb 26, 2026

@sean-freeman sean-freeman left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM - sensible update and well defined tests

@marcelmamula marcelmamula merged commit 93d8678 into sap-linuxlab:dev Feb 27, 2026
56 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants