-
Notifications
You must be signed in to change notification settings - Fork 5
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
Proposal for zApp communication #2750
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a good approach. The manifest concept provides a good foundation for future expansion while addressing immediate needs for AURA.
The authentication approach makes sense as a starting point, though I agree a proper scopes/claims system would be better long-term. For now, passing the access token to trusted apps is pragmatic.
The full-viewport iframe feature will definitely help make integrated apps feel more native within zOS.
This reverts commit a9095ad.
What does this do?
Extend the capabilities of zApps so that we can have more complex interactions between zOS and zApps
Why are we making this change?
We have a new app being built outside of zOS that'll need new capabilities
How do I test this?
Key decisions and Risk Assessment:
Things to consider:
- Sharing the original access token across apps isn't an appropriate solution for apps that are written by third parties, so we'll need to use a strict white list for what apps can use this functionality. In the future we can implement a new token system where apps can request certain permissions and the user can agree to those permissions.