Michael Salaverry
Michael Salaverry
got the same error when migrating from 9.14.5 to 9.15.1 (Node 18.7.0 on Ubuntu 22.04) once when switching the docker image version on the same sqlite db within docker-compose ```...
I believe it's related to issue #14100 and PR https://github.com/directus/directus/pull/14323 / this migration: https://github.com/directus/directus/blob/ed82a01c2c93eb74df91385b554692d2e3ea3caa/api/src/database/migrations/20220801A-update-notifications-timestamp-column.ts#L7-L10 which sqlite doesn't support due to it's lack of ALTER TABLE support to make a column...
I don't have any notifications in my sqlite db. ```sql sqlite> .schema directus_notifications CREATE TABLE IF NOT EXISTS "directus_notifications" (`id` integer PRIMARY KEY AUTOINCREMENT NOT NULL, `timestamp` datetime NOT NULL,...
I can't recreate going from 9.14.5 to 9.15.1 using a db like chinook (see my attempt to recreate repo at https://github.com/barakplasma/directus-issue-14976 ) . I also tried uploading a file before...
I have a suspicion that this bug might be caused by phantomjs failing but html-pdf not catching it and counting on there being a file when there isn't one. PR...
For a patch in the meanwhile, see https://github.com/marcbachmann/node-html-pdf/pull/644#issuecomment-1032289083 . I'm going to rollout the patch in prod, ping me later if you're interested in whether it solved the issue for...
I haven't deployed to prod yet, so no news. However, I did write a case to recreate this particular bug in [my branch](https://github.com/kvasserman/node-html-pdf/pull/1/files#diff-5bb8db779819ddef5956a5d9d5949c05ef7445237656ca37bf2f02720271440bR262-R270) which shows that the fix in #644...
@omrilit your mileage may vary, but I deployed the fix in #644 to prod, and we haven't had that error since
@kvasserman we have also been noticing failures to get the file supposedly made by phantomjs under concurrent load. I have a suspicion that the issue fixed by this PR might...
I used [patch-package](https://github.com/ds300/patch-package) to patch `[email protected]` for the project I'm working on. This patch is meant to let others use the fix in #644 until it is merged and a...