fix: update principals uri in Eventhandler.php - Githubissues

owncloud / openidconnect

OpenId Connect (OIDC) Integration for ownCloud

GNU General Public License v2.0

6 stars 2 forks source link

fix: update principals uri in Eventhandler.php #226

Closed vincentbernaud closed 2 years ago

vincentbernaud commented 2 years ago

Description

This PR fix the principals URI for sabre dav backend and allow a proper use of Caldav with OIDC access token.

Related Issue

Fixes #225

Motivation and Context

All requests to Caldav using access token failed with 404 not found. (See issue for detail)

How Has This Been Tested?

2 dev owncloud instances with different OIDC providers
Tested both basic (with app password) and bearer token authorization for a oidc-user
Caldav requests made with Postman and tsdav

Screenshots (if appropriate):

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests

Checklist:

[x] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

Open tasks:

[ ] ...

CLAassistant commented 2 years ago

All committers have signed the CLA.

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication