Skip to content

[MWPW-198657]add direct upload support to more convert verbs#827

Open
Ruchika4 wants to merge 1 commit into
stagefrom
MWPW-198657
Open

[MWPW-198657]add direct upload support to more convert verbs#827
Ruchika4 wants to merge 1 commit into
stagefrom
MWPW-198657

Conversation

@Ruchika4

Copy link
Copy Markdown
Collaborator
  • add direct upload support to more convert verbs

Resolves: MWPW-198657

Test URLs:

Testing Notes

Change: Added ppt-to-pdf, excel-to-pdf, jpg-to-pdf, png-to-pdf, and heic-to-pdf to directUploadVerbs in target-config.json. Files ≤ 1 MB on these verbs now use the single-step POST /asset/upload direct upload path instead of the create-asset → chunk → finalize chain.


For each verb (ppt-to-pdf, excel-to-pdf, jpg-to-pdf, png-to-pdf, heic-to-pdf)

Small file (≤ 1 MB) — direct upload path (new behavior)

  • Upload a valid file under 1 MB
  • Verify the upload completes and redirects to Acrobat successfully
  • Verify no finalize/polling step occurs (should be faster than before)
  • Verify the converted PDF opens correctly in Acrobat

Large file (> 1 MB) — standard chunked path (unchanged behavior)

  • Upload a valid file over 1 MB
  • Verify the upload follows the standard create-asset → chunk → finalize flow
  • Verify redirect to Acrobat and PDF opens correctly
  • Confirm no direct upload endpoint is called for large files

Regression: word-to-pdf (must be unaffected)

  • Small file (≤ 1 MB, .doc or .docx): direct upload path still works
  • Large file (> 1 MB, .doc or .docx): chunked path with finalize-await still works

Regression: other verbs (compress-pdf, pdf-to-word, etc.)

  • Verify upload and redirect behavior is unchanged for verbs not in directUploadVerbs

@aem-code-sync

aem-code-sync Bot commented Jun 19, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants