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

Missing primary keys #237

Closed
keyurshah opened this issue Jul 21, 2020 · 4 comments
Closed

Missing primary keys #237

keyurshah opened this issue Jul 21, 2020 · 4 comments
Labels
enhancement New feature or request

Comments

@keyurshah
Copy link

Hey Ben!

I'm using Craft Blitz on Digital Ocean and I'm getting an alert that they require primary keys on all their tables

I ran a diagnosis and it seems the following tables are missing a primary key

blitz_cachetags
blitz_elementcaches
blitz_elementexpirydates
blitz_elementquerycaches
blitz_elementquerysources

Can primary keys be added to these tables to avoid any future conflict with the database system.

Thanks!
@keyurshah keyurshah added the enhancement New feature or request label Jul 21, 2020
@bencroker
Copy link
Collaborator

Thanks Keyur, fixed in 01b7857 for the next release.

For reference, this is something that Digital Ocean started enforcing recently, as referenced by laravel/framework#33238 and also solved by craftcms/cms#6246.

@andrewfairlie
Copy link

Hi @bencroker thanks for this. Is there an ETA for this release, or a quick way to switch to this version pre-release?

@bencroker
Copy link
Collaborator

Most likely on Monday.

@bencroker
Copy link
Collaborator

Released in 3.6.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@keyurshah @andrewfairlie @bencroker