Skip to content

[master] Return "password fields" only if public-link password is set.#35545

Merged
patrickjahns merged 1 commit into
masterfrom
master_fix_35541
Jun 17, 2019
Merged

[master] Return "password fields" only if public-link password is set.#35545
patrickjahns merged 1 commit into
masterfrom
master_fix_35541

Conversation

@IljaN

@IljaN IljaN commented Jun 14, 2019

Copy link
Copy Markdown
Contributor

Issue: #35541
Forward port of: #35544

The iOS client uses the presence of these fields to determine if a
password is required.

Despite their names they are historically used to store pw-hashes for public-links.
@IljaN IljaN added this to the development milestone Jun 14, 2019
@IljaN IljaN self-assigned this Jun 14, 2019
@IljaN IljaN changed the title [master] Return "password fields" only if public-link password is set. #35541 [master] Return "password fields" only if public-link password is set. Jun 14, 2019
@codecov

codecov Bot commented Jun 14, 2019

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #35545      +/-   ##
============================================
+ Coverage     65.66%   65.66%   +<.01%     
- Complexity    18756    18757       +1     
============================================
  Files          1222     1222              
  Lines         70872    70873       +1     
  Branches       1289     1289              
============================================
+ Hits          46540    46541       +1     
  Misses        23954    23954              
  Partials        378      378
Flag Coverage Δ Complexity Δ
#javascript 53.7% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 67.03% <100%> (ø) 18757 <0> (+1) ⬆️
Impacted Files Coverage Δ Complexity Δ
...es_sharing/lib/Controller/Share20OcsController.php 87.39% <100%> (+0.02%) 209 <0> (+1) ⬆️

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 bd7089c...060d32a. Read the comment docs.

1 similar comment
@codecov

codecov Bot commented Jun 14, 2019

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #35545      +/-   ##
============================================
+ Coverage     65.66%   65.66%   +<.01%     
- Complexity    18756    18757       +1     
============================================
  Files          1222     1222              
  Lines         70872    70873       +1     
  Branches       1289     1289              
============================================
+ Hits          46540    46541       +1     
  Misses        23954    23954              
  Partials        378      378
Flag Coverage Δ Complexity Δ
#javascript 53.7% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 67.03% <100%> (ø) 18757 <0> (+1) ⬆️
Impacted Files Coverage Δ Complexity Δ
...es_sharing/lib/Controller/Share20OcsController.php 87.39% <100%> (+0.02%) 209 <0> (+1) ⬆️

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 bd7089c...060d32a. Read the comment docs.

@patrickjahns patrickjahns self-requested a review June 17, 2019 16:11

@patrickjahns patrickjahns left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@patrickjahns patrickjahns merged commit 038c731 into master Jun 17, 2019
@delete-merged-branch delete-merged-branch Bot deleted the master_fix_35541 branch June 17, 2019 16:12
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.

3 participants