Skip to content

Commit ae98530

Browse files
Merge pull request #55794 from nextcloud/backport/55776/stable29
[stable29] fix(TextToImage): Set better attribute for routes
2 parents 71d4cf9 + 5b851f8 commit ae98530

File tree

2 files changed

+4
-10
lines changed

2 files changed

+4
-10
lines changed

core/Controller/TextToImageApiController.php

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333
use OCP\AppFramework\Http\Attribute\ApiRoute;
3434
use OCP\AppFramework\Http\Attribute\BruteForceProtection;
3535
use OCP\AppFramework\Http\Attribute\NoAdminRequired;
36-
use OCP\AppFramework\Http\Attribute\PublicPage;
3736
use OCP\AppFramework\Http\Attribute\UserRateLimit;
3837
use OCP\AppFramework\Http\DataResponse;
3938
use OCP\AppFramework\Http\FileDisplayResponse;
@@ -69,7 +68,7 @@ public function __construct(
6968
*
7069
* 200: Returns availability status
7170
*/
72-
#[PublicPage]
71+
#[NoAdminRequired]
7372
#[ApiRoute(verb: 'GET', url: '/is_available', root: '/text2image')]
7473
public function isAvailable(): DataResponse {
7574
return new DataResponse([
@@ -90,9 +89,8 @@ public function isAvailable(): DataResponse {
9089
* 200: Task scheduled successfully
9190
* 412: Scheduling task is not possible
9291
*/
93-
#[PublicPage]
92+
#[NoAdminRequired]
9493
#[UserRateLimit(limit: 20, period: 120)]
95-
#[AnonRateLimit(limit: 5, period: 120)]
9694
#[ApiRoute(verb: 'POST', url: '/schedule', root: '/text2image')]
9795
public function schedule(string $input, string $appId, string $identifier = '', int $numberOfImages = 8): DataResponse {
9896
$task = new Task($input, $appId, $numberOfImages, $this->userId, $identifier);
@@ -126,7 +124,7 @@ public function schedule(string $input, string $appId, string $identifier = '',
126124
* 200: Task returned
127125
* 404: Task not found
128126
*/
129-
#[PublicPage]
127+
#[NoAdminRequired]
130128
#[BruteForceProtection(action: 'text2image')]
131129
#[ApiRoute(verb: 'GET', url: '/task/{id}', root: '/text2image')]
132130
public function getTask(int $id): DataResponse {
@@ -158,7 +156,7 @@ public function getTask(int $id): DataResponse {
158156
* 200: Image returned
159157
* 404: Task or image not found
160158
*/
161-
#[PublicPage]
159+
#[NoAdminRequired]
162160
#[BruteForceProtection(action: 'text2image')]
163161
#[ApiRoute(verb: 'GET', url: '/task/{id}/image/{index}', root: '/text2image')]
164162
public function getImage(int $id, int $index): DataResponse|FileDisplayResponse {

core/openapi.json

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4106,7 +4106,6 @@
41064106
"text_to_image_api"
41074107
],
41084108
"security": [
4109-
{},
41104109
{
41114110
"bearer_auth": []
41124111
},
@@ -4176,7 +4175,6 @@
41764175
"text_to_image_api"
41774176
],
41784177
"security": [
4179-
{},
41804178
{
41814179
"bearer_auth": []
41824180
},
@@ -4359,7 +4357,6 @@
43594357
"text_to_image_api"
43604358
],
43614359
"security": [
4362-
{},
43634360
{
43644361
"bearer_auth": []
43654362
},
@@ -4668,7 +4665,6 @@
46684665
"text_to_image_api"
46694666
],
46704667
"security": [
4671-
{},
46724668
{
46734669
"bearer_auth": []
46744670
},

0 commit comments

Comments
 (0)