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

feat: Make it possible to customize share link tokens #49317

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

Pytal
Copy link
Member

@Pytal Pytal commented Nov 15, 2024

Summary

  • Set custom share link tokens
  • Add admin switch

Checklist

@Pytal Pytal added this to the Nextcloud 31 milestone Nov 15, 2024
@Pytal Pytal self-assigned this Nov 15, 2024
@Pytal Pytal force-pushed the feat/edit-share-token branch from 948ea93 to 5578770 Compare December 17, 2024 00:04
@Pytal Pytal changed the title feat: Allow editing public share token feat: Make it possible to customize share link tokens Dec 17, 2024
@Pytal Pytal force-pushed the feat/edit-share-token branch from 5578770 to f57a605 Compare December 17, 2024 00:18
@Pytal Pytal marked this pull request as ready for review December 17, 2024 00:24
@Pytal Pytal requested a review from provokateurin as a code owner December 17, 2024 00:24
@Pytal Pytal added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Dec 17, 2024
@Pytal Pytal requested review from nfebe and come-nc December 17, 2024 00:25
Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

Looks mostly good to me 👍

apps/files_sharing/src/services/TokenService.ts Outdated Show resolved Hide resolved
<NcInputField v-if="config.allowCustomTokens && isPublicShare && !isNewShare"
autocomplete="off"
:label="t('files_sharing', 'Share link token')"
:helper-text="t('files_sharing', 'Set the public share link token to something easy to remember or generate a new token. It is not recommended to use a guessable token for shares which contain sensitive information.')"
Copy link
Member

Choose a reason for hiding this comment

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

@nextcloud/designers what do you think about this message? I think we should have something better that doesn't first incentivize using something easy and then reverting that in the second sentence

apps/settings/src/components/AdminSettingsSharingForm.vue Outdated Show resolved Hide resolved
core/templates/layout.public.php Outdated Show resolved Hide resolved
lib/private/Share20/Manager.php Outdated Show resolved Hide resolved
@Pytal Pytal force-pushed the feat/edit-share-token branch from f57a605 to 14bea75 Compare January 4, 2025 00:26
@Pytal Pytal requested a review from provokateurin January 4, 2025 00:26
apps/files_sharing/lib/Controller/ShareAPIController.php Outdated Show resolved Hide resolved
lib/private/Share20/Manager.php Outdated Show resolved Hide resolved
@blizzz blizzz mentioned this pull request Jan 8, 2025
@Pytal Pytal force-pushed the feat/edit-share-token branch from 14bea75 to 6bcbc5a Compare January 9, 2025 22:44
@Pytal Pytal requested a review from provokateurin January 9, 2025 22:48
@Pytal Pytal force-pushed the feat/edit-share-token branch 2 times, most recently from 7d45364 to 3801197 Compare January 9, 2025 23:17
Signed-off-by: Christopher Ng <[email protected]>
@Pytal Pytal force-pushed the feat/edit-share-token branch from 3801197 to a89f31d Compare January 9, 2025 23:25
Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

Nice!

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

Successfully merging this pull request may close these issues.

Allow to specify the token of a public share / share renaming
2 participants