Skip to content

Commit

Permalink
Removing exposed TURN ports. (#101)
Browse files Browse the repository at this point in the history
See: moby/moby#11185

Users of the Portainer frontend are publishing all exposed ports by 
default / accident.  This is causing lockups and crashes on their 
devices.  Since unexposed ports can still be published, and use of the 
internal TURN server is niche this change will save more pain than it 
will cause.  

This should be documented in a FAQ with the myriad issues Portainer can 
cause.
  • Loading branch information
felddy authored Oct 29, 2020
1 parent 70ead2b commit ea65443
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,10 @@ VOLUME ["/data"]
# HTTP Server
EXPOSE 30000/TCP
# TURN Server
EXPOSE 33478/UDP
EXPOSE 49152-65535/UDP
# Not exposing TURN ports due to bug in Docker.
# See: https://github.com/moby/moby/issues/11185
# EXPOSE 33478/UDP
# EXPOSE 49152-65535/UDP

ENTRYPOINT ["./entrypoint.sh"]
CMD ["resources/app/main.js", "--port=30000", "--headless", "--noupdate",\
Expand Down

0 comments on commit ea65443

Please sign in to comment.