Skip to content

Commit

Permalink
Merge pull request #1085 from nextcloud/url-decode-2.5
Browse files Browse the repository at this point in the history
2.5.1
  • Loading branch information
icewind1991 authored Dec 7, 2023
2 parents 6d9a2f0 + d02e1c4 commit 3e2eade
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion appinfo/info.xml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
Guests accounts can be created from the share menu by entering either the recipients email or name and choosing "create guest account", once the share is created the guest user will receive an email notification about the mail with a link to set their password.
Guests users can only access files shared to them and cannot create any files outside of shares, additionally, the apps accessible to guest accounts are whitelisted.]]></description>
<version>2.5.0</version>
<version>2.5.1</version>
<licence>agpl</licence>
<author>Nextcloud</author>
<types>
Expand Down
2 changes: 1 addition & 1 deletion lib/AppWhitelist.php
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ public function isUrlAllowed(IUser $user, $url): bool {
}

public function verifyAccess(IUser $user, IRequest $request): void {
if (!$this->isUrlAllowed($user, $request->getRawPathInfo())) {
if (!$this->isUrlAllowed($user, $request->getPathInfo())) {
header('HTTP/1.0 403 Forbidden');
Template::printErrorPage($this->l10n->t(
'Access to this resource is forbidden for guests.'
Expand Down
2 changes: 0 additions & 2 deletions lib/Controller/SettingsController.php
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,6 @@ public function setConfig(bool $useWhitelist, array $whitelist, bool $allowExter
* We do not set the whitelist to null when it is unused. This is by design.
* It allows remembering the whitelist throughout changes.
*
* @NoAdminRequired
* @return DataResponse with the current whitelist config
*/
public function getWhitelist(): DataResponse {
Expand All @@ -111,7 +110,6 @@ public function getWhitelist(): DataResponse {
/**
* AJAX handler for resetting the whitelisted apps
*
* @NoAdminRequired
* @return DataResponse with the reset whitelist
*/
public function resetWhitelist(): DataResponse {
Expand Down

0 comments on commit 3e2eade

Please sign in to comment.