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.

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

fix: wopi lock header in get lock response #10470

Merged
merged 1 commit into from
Nov 5, 2024
Merged

Conversation

micbar
Copy link
Contributor

@micbar micbar commented Nov 5, 2024

Description

Bugfix: Return wopi lock header in get lock response

We fixed a bug where the wopi lock header was not returned in the get lock response. This is now fixed and the wopi validator tests are passing.

Related Issue

  • Fixes <issue_link>

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • 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 only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@micbar micbar requested review from kobergj and jvillafanez November 5, 2024 13:20
@micbar micbar force-pushed the fix-get-lock-header branch 2 times, most recently from 3de60e7 to 8565574 Compare November 5, 2024 13:35
@micbar micbar force-pushed the fix-get-lock-header branch from 8565574 to dddc6fb Compare November 5, 2024 14:35
@micbar micbar enabled auto-merge November 5, 2024 14:44
Copy link

sonarqubecloud bot commented Nov 5, 2024

@micbar micbar merged commit 6da6c6c into master Nov 5, 2024
4 checks passed
@micbar micbar deleted the fix-get-lock-header branch November 5, 2024 15:03
ownclouders pushed a commit that referenced this pull request Nov 5, 2024
fix: wopi lock header in get lock response
This was referenced Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants