UP-NextPush / server-app

UnifiedPush provider for Nextcloud - server application - Moved to https://codeberg.org/NextPush/uppush
GNU Affero General Public License v3.0
67 stars 8 forks source link

Table \"oc_uppush_applications\" has no primary key and therefor will not behave sane in clustered setups. This will throw an exception and not be installable in a future version of Nextcloud. #34

Closed provokateurin closed 1 year ago

provokateurin commented 1 year ago

Error message appears in the server logs.

p1gp1g commented 1 year ago

Nextcloud version ?

provokateurin commented 1 year ago

25 Docker image

p1gp1g commented 1 year ago

During the app installation ?

provokateurin commented 1 year ago

No I think at every request that is made.

p1gp1g commented 1 year ago

Hm, I don't see it in my logs atm. Thanks for the report I'll look at it later

Edit: I may not have db request for a while :thinking:

provokateurin commented 1 year ago

The docker image uses sqlite by default, so that might have some effect on this.

Lastaapps commented 1 year ago

Is this error I'm getting in the admin while trying to install the UnifiedPush Provider plugin related to this issue?

25.0.4, Docker alpine, Postgres alpine 15 latest

An exception occurred while executing a query: SQLSTATE[42501]: Insufficient privilege: 7 ERROR: permission denied for schema public LINE 1: CREATE TABLE oc_uppush_applications (device_id VARCHAR(255) ... ^

Thanks

provokateurin commented 1 year ago

No that is something different. Your problem is database permissions while this is message is just a warning printed by nextcloud.

p1gp1g commented 1 year ago

@provokateurin Do you use docker-compose ? To have the same setup as you

provokateurin commented 1 year ago

No, just plain dockerfile: https://github.com/provokateurin/nextcloud-neon/blob/main/tool/Dockerfile.dev I saw the warnings in the logs when I was adding my unit tests for the API of this app.

p1gp1g commented 1 year ago

Is the error thrown when you run the tests.sh script ? Or only during the installation ? Also, you are using sqlite right ?

provokateurin commented 1 year ago

During tests, not installation and sqlite, yes.