Skip to content
This repository has been archived by the owner on Oct 20, 2022. It is now read-only.

Investigate contacts, reminders and calendars permission when looking for Java #14

Open
MagOO33 opened this issue Jan 6, 2022 · 7 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@MagOO33
Copy link

MagOO33 commented Jan 6, 2022

Hello and thank you for your work.
I want to report something strange with MultiMC 5 : when I wanted to browse to my java 8 binary in my 1.12.2 instance, MultiMC asked for an access to my contacts, my reminds and my calendars.
Is it normal ?

@ViRb3
Copy link
Member

ViRb3 commented Jan 6, 2022

Hey there. While this is definitely normal and I have also experienced it, I cannot tell you exactly why the permission prompts are showing up. You can safely deny them as none of those folders are actually used. My suspicion is that this is due to the way MultiMC (and by extension, ManyMC) searches for all Java installations. We should investigate the source and see why this is happening, though.

@ViRb3 ViRb3 changed the title security issue in ManyMC ? Investigate contacts, reminders and calendars permission when looking for Java Jan 6, 2022
@ViRb3 ViRb3 added bug Something isn't working good first issue Good for newcomers labels Jan 6, 2022
@MagOO33
Copy link
Author

MagOO33 commented Jan 7, 2022

OK, i understand. Thank you for your work.

@JJTech0130
Copy link

Perhaps if we fix this, someone should PR it in MultiMC. By the way, shouldn’t we open an upstream issue for this?

@ViRb3
Copy link
Member

ViRb3 commented Jan 29, 2022

Does this happen in MultiMC? Their release is not codesigned at all, which may bypass the permission prompts. That's not an option for M1 however.

@JJTech0130
Copy link

Wait what? They didn’t codesign it? I would have thought for sure the official version would code sign it… especially considering how much perterix hates forks 🤣

@ViRb3
Copy link
Member

ViRb3 commented Jan 29, 2022

To be honest, I don't remember. There was an issue with microphone on ManyMC because special entitlements were needed, which was not an issue on MultiMC because iirc it wasn't codesigned at all, so there were no entitlement requirements. Of course, feel free to check :)

@JJTech0130
Copy link

I believe you… just surprising

ViRb3 pushed a commit that referenced this issue Jun 6, 2022
Change flatpak to flathub
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants