X Tutup
Skip to content

Disallow sharing of share_folder and it's parents#36297

Closed
VicDeo wants to merge 1 commit intomasterfrom
fix/36241
Closed

Disallow sharing of share_folder and it's parents#36297
VicDeo wants to merge 1 commit intomasterfrom
fix/36241

Conversation

@VicDeo
Copy link
Member

@VicDeo VicDeo commented Oct 18, 2019

Description

Remove share permission for share_folder

Related Issue

Motivation and Context

It doesn't allow sharing the entire share_folder to other users/groups

How Has This Been Tested?

  1. Add share_folder into config.php
  2. User1 shares a file with User2
  3. User2 deletes the file from their account
  4. User2 shares share_folder

Actual result

share_folder can be shared. Any new files which are added to that folder as newly received shares are not shared to the collaborators. But new files can be added via public links.

Expected result

Sharing of share_folder is not allowed

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:
  • Changelog item, see TEMPLATE

@VicDeo VicDeo added this to the development milestone Oct 18, 2019
@VicDeo VicDeo self-assigned this Oct 18, 2019
@codecov
Copy link

codecov bot commented Oct 18, 2019

Codecov Report

Merging #36297 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #36297      +/-   ##
============================================
+ Coverage     64.86%   64.86%   +<.01%     
- Complexity    19775    19782       +7     
============================================
  Files          1271     1271              
  Lines         74705    74713       +8     
  Branches       1309     1309              
============================================
+ Hits          48455    48464       +9     
+ Misses        25864    25863       -1     
  Partials        386      386
Flag Coverage Δ Complexity Δ
#javascript 54% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 66.06% <100%> (ø) 19782 <3> (+7) ⬆️
Impacted Files Coverage Δ Complexity Δ
lib/private/Files/View.php 84.81% <100%> (+0.22%) 409 <3> (+7) ⬆️
lib/private/Files/Utils/Scanner.php 85.34% <0%> (ø) 39% <0%> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7bac832...f13bc98. Read the comment docs.

@VicDeo VicDeo changed the title Prevent sharing share_folder Disallow sharing of share_folder Oct 21, 2019
@VicDeo VicDeo force-pushed the fix/36241 branch 2 times, most recently from b0aaeb2 to c1b6097 Compare October 22, 2019 12:59
Copy link
Contributor

@micbar micbar left a comment

Choose a reason for hiding this comment

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

Only minor changes

@VicDeo VicDeo changed the title Disallow sharing of share_folder Disallow sharing of share_folder and it's parents Oct 22, 2019
@VicDeo VicDeo force-pushed the fix/36241 branch 4 times, most recently from bc43d08 to 2dfb528 Compare October 24, 2019 10:06
@karakayasemi
Copy link
Contributor

merged into #36337.

@karakayasemi karakayasemi deleted the fix/36241 branch October 29, 2019 14:34
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.

share_folder can be shared

3 participants

X Tutup