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

Add spreed.ME reverse proxy to Apache #64

Closed
2 tasks done
oparoz opened this issue Sep 29, 2016 · 2 comments
Closed
2 tasks done

Add spreed.ME reverse proxy to Apache #64

oparoz opened this issue Sep 29, 2016 · 2 comments
Assignees
Labels
feature New feature or request

Comments

@oparoz
Copy link
Contributor

oparoz commented Sep 29, 2016

We need to connect to the Spreed.ME Snap through a common proxy, so it's best to add it to the one used by Nextcloud. The configuration is available here:
https://github.com/strukturag/nextcloud-spreedme/blob/master/doc/example-config-apache.md

  • Add 2 new Apache modules
  • Provide configuration

This is to be added either to the default or to the custom config. There shouldn't be any harm in having a location which leads to nowhere.

# Provide access to Spreed.ME
<Location /webrtc>
    ProxyPass http://127.0.0.1:8080/webrtc
    ProxyPassReverse /webrtc
</Location>

<Location /webrtc/ws>
    ProxyPass ws://127.0.0.1:8080/webrtc/ws
</Location>

ProxyVia On
ProxyPreserveHost On
RequestHeader set X-Forwarded-Proto 'https' env=HTTPS
@oparoz oparoz added the feature New feature or request label Sep 29, 2016
@oparoz oparoz self-assigned this Sep 29, 2016
@oparoz oparoz added the high High priority label Sep 30, 2016
@oparoz oparoz added this to the 10.0.1snap1 milestone Sep 30, 2016
oparoz added a commit that referenced this issue Oct 2, 2016
Fixes #64

Signed-off-by: Olivier Paroz (oparoz) <github@oparoz.com>
@oparoz oparoz mentioned this issue Oct 2, 2016
2 tasks
@oparoz
Copy link
Contributor Author

oparoz commented Oct 2, 2016

PR is in #72

@oparoz
Copy link
Contributor Author

oparoz commented Oct 2, 2016

Documentation and test snaps are available here: https://help.nextcloud.com/t/spreed-me-snap-for-the-nextcloud-snap-beta/3757

oparoz added a commit that referenced this issue Oct 6, 2016
Fixes #64

Signed-off-by: Olivier Paroz (oparoz) <github@oparoz.com>
@kyrofa kyrofa removed this from the 10.0.1snap1 milestone Oct 13, 2016
@oparoz oparoz removed the high High priority label Oct 22, 2016
@kyrofa kyrofa closed this as completed Jan 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
No open projects
Development

No branches or pull requests

2 participants