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

webui: Dont run browser as root #5689

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adamkankovsky
Copy link
Contributor

@adamkankovsky adamkankovsky commented Jun 4, 2024

Inspirated by: #5058

This changes for liveOS was tested and works

@github-actions github-actions bot added the f41 label Jun 4, 2024
data/liveinst/liveinst Fixed Show fixed Hide fixed
@adamkankovsky adamkankovsky force-pushed the dont-run-browser-as-root-alias branch from 5b7b862 to c628fca Compare June 5, 2024 07:15
@adamkankovsky
Copy link
Contributor Author

/build-image --live

Copy link

github-actions bot commented Jun 5, 2024

Images built based on commit c628fca:

  • Live: failure

Download the images from the bottom of the job status page.

Copy link

github-actions bot commented Aug 6, 2024

This PR is stale because it has been open 60 days with no activity.
Remove stale label or comment or this will be closed in 30 days.

@github-actions github-actions bot added the stale label Aug 6, 2024
The titlebar with "Mozilla Firefox" has been fixed but there
are some more bits that can be cleaned up.

This commit achieves that by:

1. Make sure more of the environment is bubbled through anaconda to
   the webui launcher. In particular, we need XDG_CURRENT_DESKTOP, but
   this commit brings it all through, so firefox runs in an environment
   as close to getting run directly by the live user as possible.
2. Two exceptions are XAUTHORITY and XDG_RUNTIME_DIR which need to
   remain unset until we can run firefox as a normal user instead of root.
@KKoukiou KKoukiou force-pushed the dont-run-browser-as-root-alias branch from c628fca to 4cd7d0c Compare August 15, 2024 11:21
@KKoukiou
Copy link
Contributor

/build-image --live

Copy link

Images built based on commit 4cd7d0c:

  • Live: failure

Download the images from the bottom of the job status page.

@KKoukiou KKoukiou force-pushed the dont-run-browser-as-root-alias branch from edc97eb to afd2407 Compare August 15, 2024 15:12
@KKoukiou
Copy link
Contributor

/build-image --live

Copy link

Images built based on commit afd2407:

  • Live: failure

Download the images from the bottom of the job status page.

@github-actions github-actions bot removed the stale label Aug 16, 2024
@KKoukiou KKoukiou force-pushed the dont-run-browser-as-root-alias branch from afd2407 to 4cd7d0c Compare August 16, 2024 06:51
@KKoukiou
Copy link
Contributor

/build-image --live

Copy link

Images built based on commit 4cd7d0c:

  • Live: success

Download the images from the bottom of the job status page.

@KKoukiou
Copy link
Contributor

/build-image

Copy link

Images built based on commit 4cd7d0c:

  • boot.iso: success

Download the images from the bottom of the job status page.

@KKoukiou
Copy link
Contributor

KKoukiou commented Sep 5, 2024

/build-image --live

Copy link

github-actions bot commented Sep 5, 2024

Images built based on commit 4cd7d0c:

  • Live: success

Download the images from the bottom of the job status page.

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

Successfully merging this pull request may close these issues.

3 participants