Skip to content

setting the access permissions#3

Closed
jeau wants to merge 8 commits intofbartels:masterfrom
jeau:master
Closed

setting the access permissions#3
jeau wants to merge 8 commits intofbartels:masterfrom
jeau:master

Conversation

@jeau
Copy link
Copy Markdown

@jeau jeau commented Jun 7, 2018

I suggest setting access, reading and writing permissions adapted to the Clourdon context

  • export HMD_ALLOW_ANONYMOUS=false — set to allow anonymous usage (hackmd default is true)
  • export HMD_ALLOW_ANONYMOUS_EDITS=true — allowing guests to edit existing notes (hackmd default is false)
  • export HMD_ALLOW_FREEURL=false — set to allow new note creation by accessing a nonexistent note URL (hackmd default ?)
  • export HMD_DEFAULT_PERMISSION="limited" — set notes default permission (only applied on signed users)

Cf. Hackmd doc

Only the owner of the note can change the note’s permissions.

Owner read/write Signed-in read Signed-in write Guest read Guest write
Freely
Editable
Limited
Locked
Protected
Private

@jeau
Copy link
Copy Markdown
Author

jeau commented Jun 8, 2018

maybe we should set HMD_DEFAULT_PERMISSION to private to let the owner choose how to share the document and avoid some surprises

@jeau jeau closed this Jun 8, 2018
@fbartels
Copy link
Copy Markdown
Owner

fbartels commented Jun 9, 2018

Oh, it seems I missed your pr (did not get a mail notification about it), but we anyway had a similar idea.

@jeau
Copy link
Copy Markdown
Author

jeau commented Jun 9, 2018

don't worry!

I think it is also necessary to add export HMD_ALLOW_ANONYMOUS_EDITS=true so that the edition is not limited to Cloudron users. It is possible if the owner authorizes it (set Freely permission) and sends the url to an external user or publisize it.

fbartels added a commit that referenced this pull request Jun 10, 2018
Signed-off-by: Felix Bartels <felix@host-consultants.de>
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.

2 participants