Skip to content

When using audit tool, text/mediumtext columns may not be properly processed#4732

Merged
TheWitness merged 4 commits intoCacti:1.2.xfrom
ddb4github:fixaudit
Apr 20, 2022
Merged

When using audit tool, text/mediumtext columns may not be properly processed#4732
TheWitness merged 4 commits intoCacti:1.2.xfrom
ddb4github:fixaudit

Conversation

@ddb4github
Copy link
Copy Markdown
Contributor

@ddb4github ddb4github commented Apr 20, 2022

No description provided.

@ddb4github
Copy link
Copy Markdown
Contributor Author

ddb4github commented Apr 20, 2022

Reproduce:

  • alter table data_debug modify column info varchar(4096) not null
  • ./audit_database.php --alters

@ddb4github ddb4github changed the title Miss to report text/mediumtext column due to typo audit_database.php miss to report text/mediumtext column due to typo Apr 20, 2022
@TheWitness
Copy link
Copy Markdown
Member

TheWitness commented Apr 20, 2022

@ddb4github, the Audit Database has a create option audit_database.php --create. You just need to make sure that the schema is right before running. No plugins, stock install, etc.

@TheWitness TheWitness merged commit c4602ac into Cacti:1.2.x Apr 20, 2022
@ddb4github ddb4github deleted the fixaudit branch April 20, 2022 12:37
@netniV netniV changed the title audit_database.php miss to report text/mediumtext column due to typo When using audit tool, text/mediumtext columns may not be properly processed May 15, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Nov 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ddb4github @TheWitness