Skip to content

Feature/document previewer retry hardening#981

Open
KolaSailaja wants to merge 5 commits into
imDarshanGK:mainfrom
KolaSailaja:feature/document-previewer-retry-hardening
Open

Feature/document previewer retry hardening#981
KolaSailaja wants to merge 5 commits into
imDarshanGK:mainfrom
KolaSailaja:feature/document-previewer-retry-hardening

Conversation

@KolaSailaja

Copy link
Copy Markdown

📌 Summary

Closes #808

This PR hardens the retry policy used by the document previewer to improve resilience against transient failures while preserving the existing preview workflow and maintaining backward compatibility.

✨ Changes Made

  • Improved retry handling for document preview operations.
  • Strengthened error handling to retry only appropriate transient failures.
  • Preserved existing behavior for successful preview requests.
  • Prevented unnecessary or repeated retry attempts where they are not appropriate.
  • Added/updated tests covering the retry behavior and existing functionality.
  • Kept the implementation localized to minimize merge conflicts and avoid unrelated refactoring.

🧪 Testing

Validation performed includes:

  • ✅ Document preview succeeds under normal conditions.
  • ✅ Transient failures trigger the configured retry behavior.
  • ✅ Non-retryable failures are handled without unnecessary retries.
  • ✅ Existing document preview functionality remains unchanged.
  • ✅ Updated/new tests pass successfully.

📂 Files Updated

  • Document previewer implementation
  • Related backend/frontend service(s) (as applicable)
  • Relevant test files
  • Documentation or validation updates (if applicable)

🔒 Reliability & Security

  • Retry attempts are bounded to prevent infinite retry loops.
  • Existing APIs and behavior remain backward compatible.
  • No sensitive information is exposed through retry handling or logging.
  • The implementation follows the project's existing architecture and error-handling patterns.

✅ Checklist

@vercel

vercel Bot commented Jul 12, 2026

Copy link
Copy Markdown

@KolaSailaja is attempting to deploy a commit to the Darshan's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

Add retry policy hardening to document previewer

1 participant