search

truecharts / public

Community Helm Chart Repository
https://truecharts.org
GNU Affero General Public License v3.0
1.14k stars 617 forks source link

URL Preview blacklist default entires missing #9340

Closed TMTwatch closed 1 year ago

TMTwatch commented 1 year ago

App Name

Synapse

SCALE Version

22.12.2

App Version

1.82.0_7.0.32

Application Events

(it's not letting me share a screenshot in this field so here is the text)

Application Events
2023-06-02 22:02:30 Back-off restarting failed container
2023-06-02 22:02:26 Created container synapse
2023-06-02 22:02:26 Started container synapse
2023-06-02 22:02:25 Container image
"tccr.io//truecharts/synapse:1.82.0@sha256:ed16bd72a84769040c24c4931..
already present on machine
2023-06-02 22:02:10 Add eth0 [172.17.5.103/16] from ix-net
2023-06-02 22:02:10 Container image "tccr.io/truecharts/ubuntu:jammy-
20221101@sha256:4b9475e08c5180d4e7417dc6a18a26dcce7691e4311e..
already present on machine
2023-06-02 22:02:10 Created container db-wait

Application Logs

2023-06-03 02:03:54.816082+00:00 2023-06-03702:03:54.8160821207
2023-06-03 02:03:54.816116+00:00 Error in configuration:
2023-06-03 02:03:54.816119+00:00 For security, you must specify an explicit target IP address blacklist in url-preview_ip_range_blacklist for ur previe wing to work
2023-06-03 02:03:54.816149+00:00 File '/data/custom yamI' is empty or doesn't parse into a key-value map. IGNORING.

Application Configuration

Screenshot 2023-06-02 at 10 01 06 PM Screenshot 2023-06-02 at 10 04 23 PM Screenshot 2023-06-02 at 10 07 18 PM

Describe the bug

tick the Url Previews box in the application config and it will fail to deploy. A warning is given about mandatory URL blacklist IPs needed.

Official documentation on the entries https://matrix-org.github.io/synapse/latest/usage/configuration/config_documentation.html#url_preview_ip_range_blacklist

To Reproduce

get synapse server running as normal. stop application. tick URL Preview box save and start application

Expected Behavior

stuck deploying with error in log

Screenshots

Screenshot 2023-06-02 at 10 04 23 PM

Additional Context

I suspect that the default yaml has changed a lot since synapse was originally added to TC and may need to be merged again.

I've read and agree with the following

PrivatePuffin commented 1 year ago

Sadly enough, we do not accept issues for incubator apps, as they might not even work at all. Feel free to discuss on Discord and/or PR a fix.

truecharts-admin commented 1 year ago

This issue is locked to prevent necro-posting on closed issues. Please create a new issue or contact staff on discord of the problem persists