Skip to content
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

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.

@pcorless
Copy link
Owner

Just need to address these conflicts.

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.

Set tighter permissions for editing annotations
2 participants