cinnyapp / cinny

Yet another matrix client
https://cinny.in
GNU Affero General Public License v3.0
1.87k stars 242 forks source link

Cannot Set Additional Homeservers #306

Closed ericdrgn closed 2 years ago

ericdrgn commented 2 years ago

Describe the bug

I am attempting to set my own homeserver as the default from the latest release but I can't get the config.json file to actually work. The screenshot I and steps to reproduce below are from an unaltered version of cinny. While my endgoal is to get my own homeserver set the bigger problem I see is that none of the default homeservers are showing.

To Reproduce

Steps to reproduce the behavior:

  1. Download latest release
  2. Go to homeserver list
  3. no homeservers are present besides matrix.org

Expected behavior

When a config.json file is present it should have the list of homeservers which would allow me to in turn add my own and set a new default.

Screenshots

This screenshot is from the latest version and it isn't showing any other homeservers in the list with the default unchanged configuration. Screenshot from 2022-02-05 18-26-55

Desktop (please complete the following information):

ajbura commented 2 years ago

Anything useful in console?

ajbura commented 2 years ago

This only can happen when config.json isn't being served.

Sleuth56 commented 2 years ago

Same here. I'm using the docker image. The file does appear to be served. If I pull the config.json file manually nginx does return the correct data.

version: "3"
services:
  cinny:
    container_name: Cinny
    image: ajbura/cinny:latest
    hostname: cinny
    restart: unless-stopped
    #environment:
    volumes:
      - ./cinny.conf/:/app/config.json
    networks:
      - production

networks:
  production:
    external: true
Cinny    | /docker-entrypoint.sh: /docker-entrypoint.d/ is not empty, will attempt to perform configuration
Cinny    | /docker-entrypoint.sh: Looking for shell scripts in /docker-entrypoint.d/
Cinny    | /docker-entrypoint.sh: Launching /docker-entrypoint.d/10-listen-on-ipv6-by-default.sh
Cinny    | 10-listen-on-ipv6-by-default.sh: info: Getting the checksum of /etc/nginx/conf.d/default.conf
Cinny    | 10-listen-on-ipv6-by-default.sh: info: Enabled listen on IPv6 in /etc/nginx/conf.d/default.conf
Cinny    | /docker-entrypoint.sh: Launching /docker-entrypoint.d/20-envsubst-on-templates.sh
Cinny    | /docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
Cinny    | /docker-entrypoint.sh: Configuration complete; ready for start up
Cinny    | 2022/02/06 10:02:47 [warn] 1#1: duplicate extension "wasm", content type: "application/wasm", previous content type: "application/wasm" in /etc/nginx/mime.types:56
Cinny    | nginx: [warn] duplicate extension "wasm", content type: "application/wasm", previous content type: "application/wasm" in /etc/nginx/mime.types:56
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: using the "epoll" event method
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: nginx/1.21.6
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: built by gcc 10.3.1 20211027 (Alpine 10.3.1_git20211027) 
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: OS: Linux 5.4.0-94-generic
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: getrlimit(RLIMIT_NOFILE): 1048576:1048576
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: start worker processes
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: start worker process 32
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: start worker process 33
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: start worker process 34
Cinny    | 2022/02/06 10:02:47 [notice] 1#1: start worker process 35
Cinny    | 172.21.0.2 - - [06/Feb/2022:10:02:50 +0000] "GET / HTTP/1.1" 200 3084 "http://192.168.0.20.lan:80/" "Mozilla/5.0 (X11; Linux x86_64; rv:89.0) Gecko/20100101 Firefox/89.0" "192.168.0.101"
Cinny    | 172.21.0.2 - - [06/Feb/2022:10:02:50 +0000] "GET /main.e471ac6fc030ec603f53.bundle.js HTTP/1.1" 304 0 "https://example.com/" "Mozilla/5.0 (X11; Linux x86_64; rv:89.0) Gecko/20100101 Firefox/89.0" "192.168.0.101"
Cinny    | 172.21.0.2 - - [06/Feb/2022:10:02:50 +0000] "GET /polyfill.a9097dc82278a0bb6197.bundle.js HTTP/1.1" 200 97447 "https://example.com/" "Mozilla/5.0 (X11; Linux x86_64; rv:89.0) Gecko/20100101 Firefox/89.0" "192.168.0.101"
Cinny    | 172.21.0.2 - - [06/Feb/2022:10:02:51 +0000] "GET /config.json HTTP/1.1" 304 0 "https://example.com/" "Mozilla/5.0 (X11; Linux x86_64; rv:89.0) Gecko/20100101 Firefox/89.0" "192.168.0.101"

Screenshot_20220206_050904-1

ajbura commented 2 years ago

@Sleuth56 you have trailing , in homserverList after matrix.org

Sleuth56 commented 2 years ago

@ajbura That fixed it! Didn't know that json didn't support that. Thank you!

ajbura commented 2 years ago

@ericdrgn ?

kfiven commented 2 years ago

No response. Closing.. Feel free to reopen if you still have issue.