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

Add gitlab login support #47

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Add gitlab login support #47

wants to merge 15 commits into from

Conversation

mirguest
Copy link

@mirguest mirguest commented Nov 3, 2021

Release Note:

  • Allow user to use the GitLab OAuth to login the HedgeDoc

The detail implementation:

  • A helper script login-hedgedoc-via-gitlab.sh is used to wrapper the process to login via gitlab.
  • This helper script consists following part:
    • Step 1: access the callback of HedgeDoc "auth/gitlab". From the response header, the location will be retrieved.
    • Step 2: access the callback of GitLab. From the response body, the login url will be retrieved.
    • Step 3: login the GitLab. It consists two steps:
      • Step 3.1: access the login url, and get the authenticity_token.
      • Step 3.2: access the real login url and the username/password wil be posted here.
    • Step 4: access the callback of GitLab again. Then the redirection URL(HedgeDoc) will be got here.
    • Step 5: access the HedgeDoc.

Note:

  • If accessing multiple HedgeDoc servers, make sure the cookie files are different.

@mirguest mirguest force-pushed the master branch 2 times, most recently from 5e86388 to 7171e59 Compare November 4, 2021 12:56
@drupol
Copy link

drupol commented Jul 23, 2022

What's the status of this PR ?

mirguest and others added 14 commits July 23, 2022 21:50
Signed-off-by: Hugo Peixoto <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Axel Dürkop <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Axel Dürkop <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Axel Dürkop <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Axel Dürkop <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Axel Dürkop <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Co-authored-by: Erik Michelson <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
Signed-off-by: Tao Lin <[email protected]>
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.

None yet

6 participants