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

docs: gitlab hydra integration #2007 #2014

Merged
merged 12 commits into from
Sep 5, 2020

Conversation

k9ert
Copy link
Contributor

@k9ert k9ert commented Aug 24, 2020

Related issue

#2007 @aeneasr`

Proposed changes

Documenting the integration with gitlab. We're not directly using OpenID-connect in this guide but the reasons are mentioned in the appendix of gitlab.mdx.

I have not added this document to any version yet.

Checklist

  • [ X] I have read the contributing guidelines.
  • [ X ] I have read the security policy.
  • [ X ] I confirm that this pull request does not address a security
    vulnerability. If this pull request addresses a security. vulnerability, I
    confirm that I got green light (please contact
    security@ory.sh) from the maintainers to push
    the changes.
  • [ ] I have added tests that prove my fix is effective or that my feature
    works.
  • [ X] I have added or changed the documentation.

docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Show resolved Hide resolved
]
```

After you did these changes, restart the container:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NOTE TO SELF: Reviewed until here

k9ert and others added 6 commits August 25, 2020 10:59
Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
@Csirme
Copy link

Csirme commented Aug 25, 2020

I'm so happy to see that you've written the document quickly. I plan to build an internal SSO with Hydra, and I'll try to integrate it into gitlab, rancher, or others.

Thank you for your excellent product. It's meaningful

docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
docs/docs/guides/gitlab.mdx Outdated Show resolved Hide resolved
@k9ert
Copy link
Contributor Author

k9ert commented Aug 31, 2020

I've changed the whole guide to a "quick-up-and-running+troubleshooting" style, rather than a "failfast-and-fix-step-by-step" style.
Also the quickstart-gitlab.yml and the gitlab.rb is checked in and can be edited from the IDE.
Unfortunately one must slightly modify the quickstart.yml for "--dangerous-allow-insecure-redirect-urls" but i think this is acceptable over making a copy in contrib with that change only?!

Another review triggered!

@aeneasr aeneasr merged commit e2bc127 into ory:master Sep 5, 2020
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.

3 participants