Skip to content

refactor(files_external): Replace security annotations with respective attributes - #46809

Merged
provokateurin merged 1 commit into
masterfrom
refactor/files_external/security-attributes
Aug 14, 2024
Merged

refactor(files_external): Replace security annotations with respective attributes#46809
provokateurin merged 1 commit into
masterfrom
refactor/files_external/security-attributes

Conversation

@provokateurin

Copy link
Copy Markdown
Member

Summary

Follow-up to #46606

Checklist

@provokateurinprovokateurin added 3. to review Waiting for reviews technical debt 🧱 🤔🚀 ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Jul 27, 2024
@provokateurinprovokateurin added this to the Nextcloud 30 milestone Jul 27, 2024
@provokateurin
provokateurinforce-pushed the refactor/files_external/security-attributes branch from 9e5f98a to b652fccCompareJuly 27, 2024 19:33
This was referenced Jul 30, 2024
This was referenced Aug 5, 2024
@skjnldsvskjnldsv mentioned this pull request Aug 13, 2024
@skjnldsv
skjnldsv requested a review from a teamAugust 14, 2024 07:38
Comment threadapps/files_external/lib/Controller/ApiController.php Outdated
…e attributes
Signed-off-by: provokateurin <kate@provokateurin.de>
@nickvergessen
nickvergessenforce-pushed the refactor/files_external/security-attributes branch from b652fcc to c102b89CompareAugust 14, 2024 07:45
@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 14, 2024
@provokateurin
provokateurin merged commit 458524e into masterAug 14, 2024
@provokateurin
provokateurin deleted the refactor/files_external/security-attributes branch August 14, 2024 08:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finish♻️ refactorRefactor code (not a bug fix, not a feature just refactoring)technical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@provokateurin@nickvergessen@skjnldsv