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

[FEATURE REQUEST] [oCIS] Respect Capability "privateLinks": false #3732

Closed
8 of 11 tasks
michaelstingl opened this issue Aug 22, 2022 · 4 comments · Fixed by #3738
Closed
8 of 11 tasks

[FEATURE REQUEST] [oCIS] Respect Capability "privateLinks": false #3732

michaelstingl opened this issue Aug 22, 2022 · 4 comments · Fixed by #3738

Comments

@michaelstingl
Copy link
Contributor

michaelstingl commented Aug 22, 2022

Is your feature request related to a problem? Please describe.

oCIS doesn't have the oC10-style private links. It's also announced as false in the capabilities:

% cat ocis.capabilities | jq .ocs.data.capabilities.files
{
  "privateLinks": false,
  "bigfilechunking": false,
  "undelete": true,
  "versioning": true,
  "favorites": false,
  "blacklisted_files": [],
  "tus_support": {
    "version": "1.0.0",
    "resumable": "1.0.0",
    "extension": "creation,creation-with-upload",
    "max_chunk_size": 100000000,
    "http_method_override": ""
  },
  "archivers": [
    {
      "enabled": true,
      "version": "2.0.0",
      "formats": [
        "tar",
        "zip"
      ],
      "archiver_url": "/archiver",
      "max_num_files": "10000",
      "max_size": "1073741824"
    }
  ],
  "app_providers": [
    {
      "enabled": true,
      "version": "1.0.0",
      "apps_url": "/app/list",
      "open_url": "/app/open",
      "new_url": "/app/new"
    }
  ]
}

Describe the solution you'd like

Android app should respect the capability, and hide the private link entries on all screens.

Additional context

Desktop issues:

iOS issue:

TASKS

  • Research (if needed)
  • Create branch feature/private_link_capability
  • Development tasks
    • Add a new field in capabilities response
    • Add a new field in capabilities domain model
    • Add a new field in capabilities entity
    • Show or hide the private link button when sharing depending on the capability
  • Code review and apply changes requested
  • Design test plan
  • QA
  • Merge branch feature/private_link_capability into master

PR

@jesmrec
Copy link
Collaborator

jesmrec commented Aug 22, 2022

Silly question came to my mind: private link means link like http://<url>/f/<file-id> , right?

Not the internal sharing with other users (sometimes also called private sharing)

@michaelstingl
Copy link
Contributor Author

Silly question came to my mind: private link means link like http://<url>/f/<file-id> , right?

Yes exactly. http://<url>/f/<file-id> or http://<url>/index.php/f/<file-id>

@abelgardep
Copy link
Contributor

With current behavior, we don't show the private link if it is not in the files propfind.

In case the capability is false, the server shouldn't send the private link in the propfind, right?

@michaelstingl
Copy link
Contributor Author

With current behavior, we don't show the private link if it is not in the files propfind.

In case the capability is false, the server shouldn't send the private link in the propfind, right?

Yes, oCIS doesn't include the private links in the PROPFIND response.

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

Successfully merging a pull request may close this issue.

3 participants