Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Install zoomus.conf to app #143

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.

Sign up for GitHub

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

Jump to bottom
Closed
nanonyme wants to merge 4 commits into from
Closed

Install zoomus.conf to app #143

nanonyme wants to merge 4 commits into from

Conversation

@nanonyme
Copy link
Contributor

@nanonyme nanonyme commented Mar 30, 2020

This changes sandbox enough that we can install app-vendored zoomus.conf that should workaround #22

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17655

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17655 failed

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17656

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17656 failed

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17657

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17657 failed

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17658

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17658 failed

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17659

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17659 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/16920/us.zoom.Zoom.flatpakref

gasinvein
gasinvein reviewed Mar 30, 2020
View reviewed changes
@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17665

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17665 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/16926/us.zoom.Zoom.flatpakref

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17669

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17669 failed

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17671

@Arxcis Arxcis mentioned this pull request Mar 30, 2020
Closed
@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17671 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/16932/us.zoom.Zoom.flatpakref

Arxcis
Arxcis approved these changes Mar 30, 2020
View reviewed changes
Copy link

@Arxcis Arxcis left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Solved my issue 👍
Comment describing my successful test #22 (comment)

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Started test build 17672

@flathubbot
Copy link
Contributor

flathubbot commented Mar 30, 2020

Build 17672 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/16933/us.zoom.Zoom.flatpakref

@nanonyme nanonyme mentioned this pull request Mar 30, 2020
Closed
@Arxcis
Copy link

Arxcis commented Mar 30, 2020 •
edited
Loading

I tried the latest build and I can confirm that enableWaylandShare=true by default.
zoomus.conf

cat zoomus.conf 
[General]
GeoLocale=system
asBarTopMargin=30
autoPlayGif=false
bForceMaximizeWM=false
blockUntrustedSSLCert=false
captureHDCamera=true
conf.webserver=https://zoom.us
currentMeetingId=
deviceID=0C:9D:92:0F:AF:8F
embeddedBrowserForFacebookLogin=false
embeddedBrowserForGoogleLogin=false
embeddedBrowserForSSOLogin=false
enable.host.auto.grab=true
enableAlphaBuffer=true
enableCloudSwitch=false
enableLog=true
enableMiniWindow=true
enableQmlCache=true
enableScreenSaveGuard=false
enableStartMeetingWithRoomSystem=false
enableTestMode=false
enableWaylandShare=true # <--------- I did not change this manually :D
fake.version=
flashChatTime=0
forceEnableTrayIcon=true
forceSSOURL=
host.auto.grab.interval=10
isTransCoding=false
logLevel=info
newMeetingWithVideo=true
playSoundForNewMessage=false
scaleFactor=1
sso_domain=.zoom.us
sso_gov_domain=.zoomgov.com
system.audio.type=default
upcoming_meeting_header_image=
useSystemTheme=false
userEmailAddress=

[AS]
showframewindow=true

@nanonyme nanonyme mentioned this pull request Apr 12, 2020
Draft
@FakeShemp
Copy link

FakeShemp commented Apr 20, 2020

I think this would need --persist=.zoom too as Zoom tries to write stuff there.

@flathubbot
Copy link
Contributor

flathubbot commented Apr 20, 2020

Started test build 18703

@flathubbot
Copy link
Contributor

flathubbot commented Apr 20, 2020

Build 18703 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/17918/us.zoom.Zoom.flatpakref

WhyNotHugo
WhyNotHugo reviewed Apr 21, 2020
View reviewed changes
@barthalion barthalion mentioned this pull request May 21, 2020
Closed
dac73 added a commit to dac73/us.zoom.Zoom that referenced this pull request Jun 11, 2020
@dac73
closes flathub#22 flathub#18, combines flathub#182 flathub#143
e874558
@dac73 dac73 mentioned this pull request Jun 11, 2020
Closed
@TheEvilSkeleton
Copy link
Contributor

TheEvilSkeleton commented Aug 24, 2020 •
edited
Loading

Any update on this?

I've MR'd to prevent rw access to $HOME and move ~/.zoom to ~/.var/app/us.zoom.Zoom/.zoom.

I can't tell if this is MR is inactive or not, which is why I am asking. It would be cool if we could harden the sandbox first to prevent some security issues, and then fix this.

EDIT: I am a GitLab user. I use MR instead of PR. Sorry for that

@nanonyme
Copy link
Contributor Author

nanonyme commented Aug 24, 2020

This MR is pending review.

@TheEvilSkeleton
Copy link
Contributor

TheEvilSkeleton commented Aug 24, 2020

Shall I close mine?

@nanonyme
Copy link
Contributor Author

nanonyme commented Aug 24, 2020

I don't mind one way or another.

@TheEvilSkeleton TheEvilSkeleton mentioned this pull request Sep 26, 2020
Merged
@TheEvilSkeleton
Copy link
Contributor

TheEvilSkeleton commented Sep 26, 2020

Is this one working well or not?

@TheEvilSkeleton
Copy link
Contributor

TheEvilSkeleton commented Sep 26, 2020

bot, build

@flathubbot
Copy link
Contributor

flathubbot commented Sep 26, 2020

Queued test build for us.zoom.Zoom.

@flathubbot
Copy link
Contributor

flathubbot commented Sep 26, 2020

Started test build 28474

@flathubbot
Copy link
Contributor

flathubbot commented Sep 26, 2020

Build 28474 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/27412/us.zoom.Zoom.flatpakref

@TheEvilSkeleton
Copy link
Contributor

TheEvilSkeleton commented Sep 26, 2020

Doesn't seem to work for me :(

@TheEvilSkeleton TheEvilSkeleton mentioned this pull request Oct 1, 2020
Merged
@nanonyme
Copy link
Contributor Author

nanonyme commented Oct 29, 2020 •
edited
Loading

Closing in favor of #188

@nanonyme nanonyme closed this Oct 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

3 more reviewers

@WhyNotHugo WhyNotHugo WhyNotHugo left review comments

@gasinvein gasinvein gasinvein left review comments

@Arxcis Arxcis Arxcis approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@nanonyme @flathubbot @Arxcis @FakeShemp @TheEvilSkeleton @WhyNotHugo @gasinvein