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

Change default settings to be able to run ocis server without any con… #151

Merged
merged 1 commit into from
Mar 18, 2020

Conversation

IljaN
Copy link
Member

@IljaN IljaN commented Mar 18, 2020

…figuration

  • Konnectd uses no TLS as it is behind the proxy
  • Glauth generates dev-certificates for ldap on startup if none are provided,
  • Glauth can launch unencrypted (9125) and encrypted (9126) port in parallel

@IljaN IljaN requested review from butonic, micbar and refs March 18, 2020 19:39
@IljaN IljaN self-assigned this Mar 18, 2020
@update-docs
Copy link

update-docs bot commented Mar 18, 2020

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@butonic butonic force-pushed the ocis-single-binary-defaults branch 2 times, most recently from 0aec6bc to 00f9a3b Compare March 18, 2020 21:32
…figuration

- Konnectd uses no TLS as it is behind the proxy
- Glauth generates dev-certificates for ldap on startup if none are provided,
- Glauth can launch unencrypted (9125) and encrypted (9126) port in parallel

Signed-off-by: Jörn Friedrich Dreyer <jfd@butonic.de>
@butonic butonic force-pushed the ocis-single-binary-defaults branch from 00f9a3b to 2615b65 Compare March 18, 2020 21:50
Copy link
Contributor

@micbar micbar left a comment

Choose a reason for hiding this comment

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

LGTM

@butonic butonic merged commit 4c4296e into master Mar 18, 2020
@delete-merged-branch delete-merged-branch bot deleted the ocis-single-binary-defaults branch March 18, 2020 21:56
Copy link
Member

@refs refs left a comment

Choose a reason for hiding this comment

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

👍

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.

4 participants