Skip to content

mysql exception forward errorNo to SqlError#2427

Merged
an-tao merged 3 commits intodrogonframework:masterfrom
JuergenGleiss:master
Dec 25, 2025
Merged

mysql exception forward errorNo to SqlError#2427
an-tao merged 3 commits intodrogonframework:masterfrom
JuergenGleiss:master

Conversation

@JuergenGleiss
Copy link
Copy Markdown
Contributor

Till know mysql_errno is not forwarded . I've added the value to SqlError constructor.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR forwards the MySQL error number to the SqlError exception constructor, addressing the issue that mysql_errno was not previously being passed to the SqlError object. This allows error codes to be accessible from caught exceptions.

  • Updates the SqlError constructor call to include the MySQL error number
  • Switches from the 3-parameter constructor (with sqlstate) to the 4-parameter constructor (with error codes)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread orm_lib/src/mysql_impl/MysqlConnection.cc Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@an-tao an-tao merged commit b00f38f into drogonframework:master Dec 25, 2025
34 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.

3 participants