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

Need to include media_url for files M->S to work #139

Closed
olmari opened this issue Jun 11, 2019 · 1 comment
Closed

Need to include media_url for files M->S to work #139

olmari opened this issue Jun 11, 2019 · 1 comment

Comments

@olmari
Copy link

olmari commented Jun 11, 2019

If self-hosted appservice-slack is on same server than synapse is, url in config is set to localhost:port, and when media is also bridged there needs to be media_url set in appservice config file like follows (in our server):

homeserver:
  media_url: https://matrix.hacklab.fi
  url: http://localhost:8008
  server_name: hacklab.fi

Without that slackbot gives media/picture-urls as http://localhost:8008/..... to slack which is obvious failure.. But defining media_url resolves this.

ADD/EDIT: So this is first and foremost an documentation issue only, reading source code revealed this is needed and already supported and so on.

@Half-Shot
Copy link
Contributor

This got done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants