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

allow requesting capabilities from password protected links #3229

Merged
merged 1 commit into from
Feb 24, 2022

Conversation

fschade
Copy link
Contributor

@fschade fschade commented Feb 23, 2022

Description

Requesting capabilities from password protected links wasn't possible.

Related Issue

Motivation and Context

allow public links with password protection to request capabilities.

How Has This Been Tested?

  • tests added

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

@fschade fschade requested a review from C0rby February 23, 2022 17:50
@fschade fschade self-assigned this Feb 23, 2022
@ownclouders
Copy link
Contributor

💥 Acceptance test Core-API-Tests-ocis-storage-9 failed. Further test are cancelled...

@sonarcloud
Copy link

sonarcloud bot commented Feb 23, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@C0rby C0rby merged commit 479468c into master Feb 24, 2022
@delete-merged-branch delete-merged-branch bot deleted the capabilities-for-public-links branch February 24, 2022 08:43
ownclouders pushed a commit that referenced this pull request Feb 24, 2022
Merge: e92e84d e870208
Author: David Christofas <dchristofas@owncloud.com>
Date:   Thu Feb 24 09:43:36 2022 +0100

    Merge pull request #3229 from owncloud/capabilities-for-public-links

    allow requesting capabilities from password protected links
@micbar micbar mentioned this pull request Mar 3, 2022
26 tasks
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