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.

Sign up for GitHub

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

Jump to bottom

Unable to convert database to latin1 instead of utf8 if desired #5450

Closed
TheWitness opened this issue Aug 12, 2023 · 4 comments
Closed

Unable to convert database to latin1 instead of utf8 if desired #5450

TheWitness opened this issue Aug 12, 2023 · 4 comments
Labels
bug Undesired behaviour resolved A fixed issue
Milestone
v1.2.25

Comments

@TheWitness
Copy link
Member

Describe the bug

There is presently no way to mass convert tables to latin1 if you are space constrained and don't use international characters.

To Reproduce

Steps to reproduce the behavior:

  1. Go to cli folder

  2. Run ./convert_tables.php --help

  3. There is no latin1 option

Expected behavior

Cacti should provide the ability to move between utf8mb4 and latin1
@TheWitness TheWitness added the bug Undesired behaviour label Aug 12, 2023
@TheWitness TheWitness added this to the v1.2.25 milestone Aug 12, 2023
TheWitness added a commit that referenced this issue Aug 12, 2023
@TheWitness
Issue #5450 - Bulk convert to latin1
ea05cd5 
* Unable to bulk convert to latin1 if desired
* Minor paranoa on audit_database.php
@TheWitness TheWitness added the resolved A fixed issue label Aug 12, 2023
TheWitness added a commit that referenced this issue Aug 12, 2023
@TheWitness
Issue #5450 - Bulk convert to latin1
8746a05 
* Unable to bulk convert to latin1 if desired
* Minor paranoa on audit_database.php
@netniV
Copy link
Member

netniV commented Aug 12, 2023

It should be noted, this will make the installer request table conversion each time until utf8 is restored.

@TheWitness
Copy link
Member Author

Yea, maybe we should have set the character set for the whole tables and verify against that.

@netniV
Copy link
Member

netniV commented Aug 13, 2023

No I believe the default of UTF8's is correct for modern dbs

@TheWitness
Copy link
Member Author

We will have to address as a feature in 1.3.

@netniV netniV changed the title Unable to bulk convert to latin1 if desired Unable to convert database to latin1 instead of utf8 if desired Sep 3, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Dec 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Undesired behaviour resolved A fixed issue
Projects
None yet
Milestone
v1.2.25
Development

No branches or pull requests
2 participants
@TheWitness @netniV