Skip to content

Comments

This commit fixes error when updating multiple confluence attachments#13341

Merged
cderv merged 8 commits intoquarto-dev:mainfrom
fkgruber:bug/confluence500error
Feb 18, 2026
Merged

This commit fixes error when updating multiple confluence attachments#13341
cderv merged 8 commits intoquarto-dev:mainfrom
fkgruber:bug/confluence500error

Conversation

@fkgruber
Copy link
Contributor

@fkgruber fkgruber commented Sep 6, 2025

Description

This PR fixes the problem described in #12558. Essentially you can't update more than 1 attachment at a time because you get a 500 error. The issue appears to be fixed if you upload attachments one at a time on a for loop with a delay im between.

Checklist

I have (if applicable):

  • [ X] filed a contributor agreement.
  • [ X] referenced the GitHub issue this PR closes
  • updated the appropriate changelog in the PR
  • ensured the present test suite passes
  • added new tests
  • created a separate documentation PR in Quarto's website repo and linked it to this PR

@posit-snyk-bot
Copy link
Collaborator

posit-snyk-bot commented Sep 6, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@cderv cderv self-requested a review October 2, 2025 14:24
@cscheid
Copy link
Collaborator

cscheid commented Feb 17, 2026

@cderv You added yourself as a reviewer. I think this is good to merge (there might be a better way that avoids the first delay if there's a single attachment, but I think this is fine). We do need a changelog entry, though.

@cscheid cscheid added this to the v1.9 milestone Feb 17, 2026
@cderv cderv force-pushed the bug/confluence500error branch from 0227f9b to 9d7f46e Compare February 18, 2026 16:40
Copy link
Collaborator

@cderv cderv left a comment

Choose a reason for hiding this comment

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

Thanks ! I rebased on main, and adapted a few things.

I'll merge today

@cderv cderv merged commit eef6d94 into quarto-dev:main Feb 18, 2026
51 checks passed
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.

4 participants