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

Graph API permission #21

Open
Ivanodib opened this issue Oct 31, 2024 · 8 comments
Open

Graph API permission #21

Ivanodib opened this issue Oct 31, 2024 · 8 comments

Comments

@Ivanodib
Copy link

Upload doesn't work. Is App registration required? which permissions are needed? delegated or application? If needed, how can i import ClientID and ClientSecret ?

@Metisak
Copy link

Metisak commented Nov 1, 2024

I have probaly similiar issue...
image

@tori321
Copy link

tori321 commented Nov 3, 2024

Upload doesn't work. Is App registration required? which permissions are needed? delegated or application? If needed, how can i import ClientID and ClientSecret ?

Microsoft has complicated this early this year by terminating Microsoft intune Powershell enterprise application.

instead you need to create an app registration yourself.
Follow this article
https://techcommunity.microsoft.com/t5/intune-customer-success/update-to-microsoft-intune-powershell-example-script-repository/ba-p/3842452

@huuub
Copy link

huuub commented Dec 9, 2024

So the app is not working at the moment... Correct?

@Ivanodib
Copy link
Author

Ivanodib commented Dec 9, 2024

So the app is not working at the moment... Correct?

Correct. I found https://github.com/Romanitho/WingetIntunePackager , the last PR includes App registration Id field. (more info at pull request Romanitho/WingetIntunePackager#30

Not tried yet, let me know if this works

@huuub
Copy link

huuub commented Dec 9, 2024

Well... I tried that one before yours. With that one I cannot even connect. Even after creating an app in entra and giving the right permissions and the uri. So that's why I gave this one a try. I guess I am out of options.

@Ivanodib
Copy link
Author

Ivanodib commented Dec 9, 2024

Well, i'm gonna create it on my own :).

@InnovationForge-com
Copy link
Contributor

InnovationForge-com commented Dec 9, 2024

I fixed the issues which occured after the MS App Registration change within the script and added as well to the GUI to set your own non default Client ID and Redirect URI. If the app registration doesn't have the permission, you will be asked with Connect-MgGraph to give the permission.
image

You can get my version in my fork: https://github.com/InnovationForge-com/WinGet-Wrapper
I will create a pull request after optimizing it a bit more.

If you want the GUI options then you need to start WinGet-WrapperImportGUI.ps1 and not WinGet-WrapperImportGUI.exe.
Even without setting it in the GUI, WinGet-WrapperImportFromCSV.ps1 will use App ID "14d82eec-204b-4c2f-b7e8-296a70dab67e" and Redirect URI "https://login.microsoftonline.com/common/oauth2/nativeclient", which is used by the .exe as well.

@SorenLundt
Copy link
Owner

SorenLundt commented Dec 9, 2024

I fixed the issues which occured after the MS App Registration change within the script and added as well to the GUI to set your own non default Client ID and Redirect URI. If the app registration doesn't have the permission, you will be asked with Connect-MgGraph to give the permission.

image

You can get my version in my fork: https://github.com/InnovationForge-com/WinGet-Wrapper

I will create a pull request after optimizing it a bit more.

If you want the GUI options then you need to start WinGet-WrapperImportGUI.ps1 and not WinGet-WrapperImportGUI.exe.

Even without setting it in the GUI, WinGet-WrapperImportFromCSV.ps1 will use App ID "14d82eec-204b-4c2f-b7e8-296a70dab67e" and Redirect URI "https://login.microsoftonline.com/common/oauth2/nativeclient", which is used by the .exe as well.

Great work!
Feel free to create a pull request and i will be sure to test it and add it to the main branch.

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

No branches or pull requests

6 participants