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.

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

[BUG] default site-conf is broken #403

Closed
1 task done
Gamer92000 opened this issue Jan 2, 2024 · 9 comments · Fixed by #405
Closed
1 task done

[BUG] default site-conf is broken #403

Gamer92000 opened this issue Jan 2, 2024 · 9 comments · Fixed by #405
Assignees
Labels
work-in-progress Stale exempt

Comments

@Gamer92000
Copy link

Gamer92000 commented Jan 2, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

The changes to the default site conf introduced in 35de5f8 prohibit some files from loading properly. Notably, /js/core/merged-template-prepend.js never loaded for me, resulting in a 404.

Expected Behavior

Files like /js/core/merged-template-prepend.js properly load.

Steps To Reproduce

  1. Setup the container
  2. Go to /js/core/merged-template-prepend.js.
  3. Get a 404

Environment

- OS: Ubuntu 22.04.3 LTS x86_64
- How docker service was installed: from the official Docker repos, version `5:24.0.7-1~ubuntu.22.04~jammy`

CPU architecture

x86-64

Docker creation

version: "3"

services:
  nextcloud:
    image: lscr.io/linuxserver/nextcloud:latest
    container_name: nextcloud
    environment:
      - TZ=Europe/Berlin
    volumes:
      - /jbod/nextcloud/config:/config
      - /jbod/nextcloud/data:/data
    restart: unless-stopped
    labels:
      traefik.enable: true
      [...]
    networks:
      - traefik
      - default

Container logs

[migrations] started
[migrations] 01-nginx-site-confs-default: skipped
[migrations] 02-default-location: skipped
[migrations] done
usermod: no changes
───────────────────────────────────────

      ██╗     ███████╗██╗ ██████╗ 
      ██║     ██╔════╝██║██╔═══██╗
      ██║     ███████╗██║██║   ██║
      ██║     ╚════██║██║██║   ██║
      ███████╗███████║██║╚██████╔╝
      ╚══════╝╚══════╝╚═╝ ╚═════╝ 

   Brought to you by linuxserver.io
───────────────────────────────────────

To support LSIO projects visit:
https://www.linuxserver.io/donate/

───────────────────────────────────────
GID/UID
───────────────────────────────────────

User UID:    911
User GID:    911
───────────────────────────────────────

using keys found in /config/keys
[custom-init] No custom files found, skipping...
[ls.io-init] done.

Nginx error log

2024/01/02 05:23:02 [error] 370#370: *1 open() "/app/www/public/js/core/merged-template-prepend.js" failed (2: No such file or directory), client: <redacted>, server: _, request: "GET /js/core/merged-template-prepend.js HTTP/2.0", host: <redacted>
Copy link

github-actions bot commented Jan 2, 2024

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

@ownedge13
Copy link

ownedge13 commented Jan 2, 2024

i have the same issue when i go in the shell of the docker the merged-template-prepend.json is in /app/www/public/core/js
but it search it in
2024/01/02 13:21:57 [error] 420#420: *16 open() "/app/www/public/js/core/merged-template-prepend.js" failed (2: No such file or directory)

iirc i'm stuck with that error starting the 25 december

edit : i have reverted to the 2023/06/23 default site-conf file and it solve my problem, my public shares link are now working again

@RabbITCybErSeC
Copy link

RabbITCybErSeC commented Jan 2, 2024

I am having the same issue here after upgrading to the new default.conf:

Error logs in the nginx logs:

2024/01/02 15:23:57 [error] 412#412: *245 open() "/app/www/public/js/core/merged-template-prepend.js" failed (2: No such file or directory), client: 192.168.178.21, server: _, request: "GET /js/core/merged-template-prepend.js?v=c5940ea3-19 HTTP/1.1", host: "removed"

Temp fix:

I fixed the issue by reverting back to the previous default.conf of release https://github.com/linuxserver/docker-nextcloud/releases/tag/27.1.4-ls286. You can just copy paste over the previous default.conf.

@drizuid
Copy link
Member

drizuid commented Jan 2, 2024

we have a PR being confirmed now to resolve this. applies to #402 also

@drizuid drizuid added the work-in-progress Stale exempt label Jan 2, 2024
This was referenced Jan 2, 2024
@nemchik nemchik closed this as completed Jan 2, 2024
@LinuxServer-CI LinuxServer-CI moved this from Issues to Done in Issue & PR Tracker Jan 2, 2024
@ownedge13
Copy link

ownedge13 commented Jan 3, 2024

i'm sorry to bother or maybe it's something else but with the new default.conf i still got the error
2024/01/03 13:53:51 [error] 392#392: *2448 open() "/app/www/public/js/core/merged-template-prepend.js" failed (2: No such file or directory)

and with the old default.conf (i use the 2023/06/23 one) it work

if you look the error it search /app/www/public/js/core/merged-template-prepend.js

but in the docker the file is in /app/www/public/core/js/merged-template-prepend.json

@drizuid
Copy link
Member

drizuid commented Jan 3, 2024

you guys are correct, its not fixed, im working on a new PR now. sorry!

@drizuid drizuid reopened this Jan 3, 2024
@drizuid drizuid self-assigned this Jan 3, 2024
@LinuxServer-CI LinuxServer-CI moved this from Done to Issues in Issue & PR Tracker Jan 3, 2024
@drizuid
Copy link
Member

drizuid commented Jan 3, 2024

mr pr is here, but isn't covering all the issues yet. you're still welcome to play with it. #405

@drizuid
Copy link
Member

drizuid commented Jan 3, 2024

hi guys, please test and report back
lspipepr/nextcloud:amd64-28.0.1-pkg-1497da69-dev-4030be827853102c904c898015197f9583a2f5fb-pr-405

@drizuid
Copy link
Member

drizuid commented Jan 3, 2024

to avoid this affecting more and more people, since we have consensus from an affected user in this ghi (own) and from the other thread related to the same issue, ill consider this resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
work-in-progress Stale exempt
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

5 participants