search

Autonomous-Motorsports-Purdue / AMP_ASSv2

Reassembled, simpler, without move_base and loop closure issues
5 stars 1 forks source link

fix: enabled xhost #36

Closed alanssitis closed 2 years ago

alanssitis commented 2 years ago

What is a quick description of the change?

Disable xhost access control in order for the container to access X11 display.

Is this fixing an issue?

No

Were any issues created as a result of this change?

No

Are there more details that are relevant?

No

Any additional comments?

Reschivon commented 2 years ago

I think you can enable xhost for docker only. Right now anyone can access the xserver while the container is running. xhost +local:docker

alanssitis commented 2 years ago

Recommendation implemented!