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

Login/App Permissions #134

Open
PJMarcum opened this issue May 5, 2023 · 6 comments
Open

Login/App Permissions #134

PJMarcum opened this issue May 5, 2023 · 6 comments

Comments

@PJMarcum
Copy link

PJMarcum commented May 5, 2023

  1. The app shouldn't need Read/Write to simply document. Getting approval for Read/Write is much harder that getting approval for simply reading.
  2. When using an existing app registration and secret key login shouldn't be required.
@Micke-K
Copy link
Owner

Micke-K commented May 6, 2023

Hello!

The app doesn't require write. If you use it to request consent, they it will add readwrite. You can set up your own app with just read permissions. Menu items will then be orange because of limited access. The readwrite default consent permissions is because that is the main functionality of the app eg export/import but again that doesn't stop you from using custom permissions. I use two apps, one for read and one for write.

Regarding app login, I'll see if I can have a look at this for next version. There is another request for this. You can do it in the batch job but not in the UI. I have to re-write some UI/Authentication parts to support it.

Cheers!

@Micke-K
Copy link
Owner

Micke-K commented May 16, 2023

Hello!

What do you get when connecting to an app with read only access?

The app should warn you about it and menu items should be orange but that's it. This should be the default behaviour.

There is a setting where the app will display a consent prompt if you don't have all permissions for full functionality. This happens if you turn off Use Default Permissions.

I'll see if I can test it this week.

Cheers!

@Micke-K
Copy link
Owner

Micke-K commented May 30, 2023

Hello

I created an Azure App with read only delegated permissions and logged on with my user and it is working. All items in the menu that has read permissions are Orange and those without permissions are Red. I could also generate a documentation based on the policies. I will look into supporting authenticating with the app as well.

image

@PJMarcum
Copy link
Author

PJMarcum commented May 30, 2023 via email

@Micke-K
Copy link
Owner

Micke-K commented May 30, 2023

Hello!

Check in your Settings. Do you have "Use Default Permissions" unchecked?

If this is not checked, it will prompt for missing permissions. If it is checked, it will just use the permissions assigned to the app.

Cheers!

@Micke-K
Copy link
Owner

Micke-K commented Oct 21, 2023

Checking in to see if we can close this issue.

Cheers!

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

2 participants