Skip to content

Av/327 share resources with inheritance#389

Merged
anushkavidanage merged 11 commits intodevfrom
av/327_share_resources_with_inheritance
Oct 22, 2025
Merged

Av/327 share resources with inheritance#389
anushkavidanage merged 11 commits intodevfrom
av/327_share_resources_with_inheritance

Conversation

@anushkavidanage
Copy link
Collaborator

@anushkavidanage anushkavidanage commented Oct 16, 2025

Pull Request Details

What issue does this PR address

Testing

  • Please use Add/Delete Permission from Any Resource button to test this function.

Checklist

Complete the check-list below to ensure your branch is ready for PR.

Flutter Style Guide: https://survivor.togaware.com/gnulinux/flutter-style.html

  • Screenshots included in linked issue
  • Changes adhere to the team style and coding guideline
  • No confidential information
  • No duplicated content
  • No lint check errors related to your changes (make prep or flutter analyze lib)
  • Pre-exisiting lint errors noted: [HERE]
  • Tested on at least one device
    • Android Phone
    • Android Emulator
    • Chrome on Android
    • Chrome
    • iOS
    • Linux
    • MacOS
    • Windows
  • Added 2 reviewers (or 1 for private repositories then they add another)

Finalising

Once PR discussion is complete and 2 reviewers have approved:

  • Merge dev into the branch
  • Resolve any conflicts
  • Add one line summary into CHANGELOG.md
  • Bump appropriate version number in pubspec.yaml
  • Push to git repository and review
  • Merge PR into dev

Copy link
Collaborator

@cdawei cdawei left a comment

Choose a reason for hiding this comment

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

It seems a folder can be shared in the demo app in my test, thanks Anushka.
image

But I haven't figured out how to check the sharing from the recipient, as clicking the button View all Resources your WebID have access to shows permissions only for key-value.ttl. Is there another way to test this?
image

@cdawei
Copy link
Collaborator

cdawei commented Oct 20, 2025

I can see a log entry in the recipient's logs/permissions-log.ttl about the shared directory, and can also find triples in shared/shared-keys.ttl presumably of the corresponding (encrypted) individual key. Thanks Anushka.

@srodriguez142857
Copy link
Collaborator

srodriguez142857 commented Oct 20, 2025

I get the following error when removing a granted permission for a resource directory:

image
[ERROR:flutter/runtime/dart_vm_initializer.cc(40)] Unhandled Exception: 'package:solidpod/src/solid/api/rest_api.dart': Failed assertion: line 275 pos 12: 'resUrl.endsWith('/')': is not true.
#0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:67:4)
#1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:49:5)
#2      checkResourceStatus (package:solidpod/src/solid/api/rest_api.dart:275:12)
#3      removeSharedKey (package:solidpod/src/solid/api/revoke_permission_api.dart:151:13)
<asynchronous suspension>
#4      revokePermission (package:solidpod/src/solid/revoke_permission.dart:123:15)
<asynchronous suspension>
#5      buildPermDataTable.<anonymous closure>.<anonymous closure>.<anonymous closure>.<anonymous closure> (package:solidpod/src/widgets/file_permission_data_table.dart:133:31)
<asynchronous suspension>

Additionally, I don't see any updates as Dawei indicated above (#389 (comment)).

@anushkavidanage
Copy link
Collaborator Author

@srodriguez142857 I've pushed some changes to fix the issue you had. Please have another look and let me know if you see any other issues.

@srodriguez142857
Copy link
Collaborator

Hi @anushkavidanage.
I did a round of tests. All good. I get the expected behaviour as Dawei described above.
Thanks.

Copy link
Collaborator

@srodriguez142857 srodriguez142857 left a comment

Choose a reason for hiding this comment

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

The functionality is working as expected.

@anushkavidanage anushkavidanage merged commit 829021e into dev Oct 22, 2025
14 of 20 checks passed
@anushkavidanage anushkavidanage deleted the av/327_share_resources_with_inheritance branch October 22, 2025 10:00
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

Successfully merging this pull request may close these issues.

3 participants

Comments