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.

Sign up for GitHub

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

Jump to bottom

fix: Let wopi middleware also handle asset fetching #3403

Merged
merged 2 commits into from
Jan 11, 2024
Merged

fix: Let wopi middleware also handle asset fetching #3403

merged 2 commits into from
Jan 11, 2024

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented Jan 11, 2024 •
edited

  • fix: Let wopi middleware also handle asset fetching
  • chore: Move to class/method attributes for middleware checks

@juliushaertl
fix: Let wopi middleware also handle asset fetching
7edf59f 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl juliushaertl added this to the Nextcloud 29 milestone Jan 11, 2024
@juliushaertl juliushaertl added bug Something isn't working 3. to review Ready to be reviewed technical debt labels Jan 11, 2024
@juliushaertl juliushaertl force-pushed the bugfix/middleware-asset branch 2 times, most recently from 2bd7e4a to 19a2aa2 Compare January 11, 2024 10:38
max-nextcloud
max-nextcloud requested changes Jan 11, 2024
View reviewed changes
Copy link
Contributor

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Either I'm confused or this would not work as intended. 😉

lib/Middleware/WOPIMiddleware.php Outdated Show resolved Hide resolved
lib/Middleware/WOPIMiddleware.php Outdated Show resolved Hide resolved
@juliushaertl
chore: Move to class/method attributes for middleware checks
93225dd 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
max-nextcloud
max-nextcloud approved these changes Jan 11, 2024
View reviewed changes
Copy link
Contributor

@max-nextcloud max-nextcloud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for addressing my comments so fast... Now looks good to me.

@juliushaertl juliushaertl merged commit 745b10a into main Jan 11, 2024
58 checks passed
@juliushaertl juliushaertl deleted the bugfix/middleware-asset branch January 11, 2024 11:49
@juliushaertl
Copy link
Member Author

/backport to stable28

@juliushaertl
Copy link
Member Author

/backport 7edf59f to stable27

@juliushaertl
Copy link
Member Author

/backport 7edf59f to stable26

@juliushaertl
Copy link
Member Author

/backport 7edf59f to stable25

@juliushaertl
Copy link
Member Author

/backport 7edf59f to stable24

@juliushaertl
Copy link
Member Author

/backport 7edf59f to stable23

This was referenced Jan 11, 2024
Merged
Merged
Merged
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@luka-nextcloud luka-nextcloud luka-nextcloud approved these changes

@max-nextcloud max-nextcloud max-nextcloud approved these changes

@blizzz blizzz Awaiting requested review from blizzz

@mejo- mejo- Awaiting requested review from mejo-

Assignees
No one assigned
Labels
3. to review Ready to be reviewed bug Something isn't working technical debt
Projects
None yet
Milestone
Nextcloud 29
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@juliushaertl @luka-nextcloud @max-nextcloud