Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

GH-254 Set tighter permissions #255

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

gtache
Copy link
Contributor

@gtache gtache commented Feb 13, 2023

Closes #254
Should we add an option / a flag to explicitly enable this option?
Also depends on #253

@pcorless
Copy link
Owner

I've given the PDF spec a good look over and I can't find anything specific to user permissions and annotation edits. There is an annotation flag for read-only but I don't see that being applicable here. I'm currently leaning towards not putting this behind a feature flag. If there is any push back from other library users, it can be reconsidered.

@gtache gtache force-pushed the modifyPermissions branch from a639132 to d89e2bf Compare July 25, 2023 10:46
@pcorless
Copy link
Owner

Just need to address these conflicts.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set tighter permissions for editing annotations
2 participants