Skip to content

[*] fix updateSources() query parameters number#875

Merged
pashagolub merged 1 commit intocybertec-postgresql:masterfrom
0xgouda:fix-update-srcs
Jul 31, 2025
Merged

[*] fix updateSources() query parameters number#875
pashagolub merged 1 commit intocybertec-postgresql:masterfrom
0xgouda:fix-update-srcs

Conversation

@0xgouda
Copy link
Copy Markdown
Collaborator

@0xgouda 0xgouda commented Jul 31, 2025

  • Change number of sql query parameters in updateSources() to be 13 not 14

After removing host_config from sources in #859 updateSources() sql query should take 13 parameters not 14.
currently any sources update gives the following error:

[err:ERROR: INSERT has more expressions than target columns (SQLSTATE 42601)] [pid:39828] [time:686.266µs] Query

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 16638151836

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 63.974%

Totals Coverage Status
Change from base Build 16628638577: 0.0%
Covered Lines: 3191
Relevant Lines: 4988

💛 - Coveralls

@pashagolub pashagolub self-assigned this Jul 31, 2025
@pashagolub pashagolub added bug Something isn't working sources What sources and in what way to monitor labels Jul 31, 2025
@pashagolub pashagolub merged commit bfad57a into cybertec-postgresql:master Jul 31, 2025
1 check passed
@0xgouda 0xgouda deleted the fix-update-srcs branch August 1, 2025 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working sources What sources and in what way to monitor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants