-
-
Notifications
You must be signed in to change notification settings - Fork 174
feat: iOS OCR Server with hOCR support + preserve document owner, permissions and document_type #973
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
Open
vistalba
wants to merge
26
commits into
icereed:main
Choose a base branch
from
vistalba:feat/ios-ocr-server-hocr
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
feat: iOS OCR Server with hOCR support + preserve document owner, permissions and document_type #973
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
d3fa493
fix: preserve owner and document_type when replacing document in uplo…
d44450d
fix: preserve full owner and permissions via PATCH after document upload
949ce8a
feat: add iOS OCR Server provider
3d14a8c
fix: update Alpine package versions in Dockerfile
8c0862a
fix: remove Alpine version pins to fix build
f7c2425
docs: add iOS OCR Server to README
d23634f
Revert "fix: remove Alpine version pins to fix build"
9912d85
chore: restore original Dockerfile from upstream
50e16af
docs: add App Store link for iOS OCR Server
48306ba
fix: validate timeout > 0 and use bounded reader for iOS OCR responses
68b065c
fix: replace retryablehttp with plain http.Client to avoid duplicate …
0582478
feat: add hOCR support for ios_ocr provider
b89f818
fix: resolve variable shadowing in ocr.go image mode branch
65102b7
docs: update Enhanced OCR Features section for iOS OCR Server hOCR su…
04dca66
fix: address code review feedback - brittle test string, nil guard in…
8248d97
fix: add PatchDocument to ClientInterface and mock implementations
2b57f0a
fix: decode GetTaskStatus response as array, extract document ID from…
f6a3846
feat: add PDF_PRESERVE_OWNER_PERMISSIONS env var to preserve document…
b4eeee8
feat: async in-memory queue for pending permission restores
a0820a8
fix: use correct Permissions type (not PermissionSet) in queue struct…
8d1f665
docs: add PDF_PRESERVE_OWNER_PERMISSIONS to README with trade-off note
25e5689
style: gofmt
1fa994d
fix: address CodeRabbit review findings
957f65d
fix: guard ReplaceOriginal permission restore with PreserveOwnerPermi…
19fdec5
fix: increase ReplaceOriginal polling timeout to ~1 hour
d486e45
fix: only delete original document on confirmed SUCCESS
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.