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

When importing packages, SQL errors may be generated #5291

Closed
jdcoats opened this issue Apr 4, 2023 · 7 comments
Closed

When importing packages, SQL errors may be generated #5291

jdcoats opened this issue Apr 4, 2023 · 7 comments
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue unverified Some days we don't have a clue
Milestone

Comments

@jdcoats
Copy link

jdcoats commented Apr 4, 2023

2023/04/04 14:59:08 - CMDPHP SQL Backtrace: (/package_import.php[41]:form_save(), /package_import.php[188]:import_package(), /lib/import.php[587]:import_xml_data(), /lib/import.php[235]:xml_to_data_template(), /lib/import.php[1179]:db_execute_prepared())
2023/04/04 14:59:08 - CMDPHP ERROR: A DB Exec Failed!, Error: You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'data_template_rrd AS dtr ON gti.task_item_id = dtr.id INNER JOIN data_input_f...' at line 1
@jdcoats jdcoats added bug Undesired behaviour unverified Some days we don't have a clue labels Apr 4, 2023
@TheWitness
Copy link
Member

Your running 1.2.x branch?

@jdcoats
Copy link
Author

jdcoats commented Apr 4, 2023

Yes, saw the same thing running the upgrade to 1.2.25 yesterday also. Should have mentioned that today

@jdcoats
Copy link
Author

jdcoats commented Apr 5, 2023

I will update to develop if you have time to follow through with me to stabilization :)

@TheWitness
Copy link
Member

Not yet Dan. There is some heavy lifting yet to do. If you want to do a dev setup, that fine.

@jdcoats
Copy link
Author

jdcoats commented Apr 5, 2023

Thought so, it would be some work here also as every device has an ACL allowing snmp query so everything would have to be touched to allow that and 2x query’s every minute would be excessive also

@TheWitness
Copy link
Member

This is resolved now.

@TheWitness
Copy link
Member

Fixed here.

TheWitness added a commit that referenced this issue Apr 6, 2023
Error thrown when importing packages
TheWitness added a commit that referenced this issue Apr 6, 2023
@TheWitness TheWitness added this to the v1.3.0 milestone Apr 6, 2023
@TheWitness TheWitness added confirmed Bug is confirm by dev team resolved A fixed issue labels Apr 6, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Jul 6, 2023
@netniV netniV changed the title Error thrown when installing packages When importing packages, SQL errors may be generated Sep 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Undesired behaviour confirmed Bug is confirm by dev team resolved A fixed issue unverified Some days we don't have a clue
Projects
None yet
Development

No branches or pull requests

2 participants