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

SASL support #643

Merged
merged 4 commits into from
Aug 10, 2018
Merged

SASL support #643

merged 4 commits into from
Aug 10, 2018

Conversation

Half-Shot
Copy link
Contributor

This should hopefully fix #470

@Half-Shot Half-Shot requested a review from a team August 10, 2018 11:01
@Half-Shot Half-Shot changed the base branch from master to develop August 10, 2018 11:01
@Half-Shot Half-Shot changed the title Hs/sasl support SASL support Aug 10, 2018
@@ -63,6 +63,9 @@ ircService:
ssl: true
# Whether or not IRC server is using a self-signed cert or not providing CA Chain
sslselfsign: false
# Should the connection attempt to identify via SASL (if a server or user password is given?)
Copy link
Member

Choose a reason for hiding this comment

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

question should be answered

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure why there is a ?. It reads fine without.

Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

I'm assuming this has been tested, and doesn't cause explosions.

@Half-Shot Half-Shot merged commit 56569f1 into matrix-org:develop Aug 10, 2018
eqyiel added a commit to eqyiel/matrix-appservice-irc-nixos that referenced this pull request Sep 11, 2018
eqyiel added a commit to eqyiel/matrix-appservice-irc-nixos that referenced this pull request Sep 11, 2018
eqyiel added a commit to eqyiel/matrix-appservice-irc-nixos that referenced this pull request Sep 11, 2018
@eqyiel
Copy link

eqyiel commented Sep 20, 2018

@Half-Shot would you happen to know anything about this (kind of related) issue? #689

Ralith pushed a commit to Ralith/matrix-appservice-irc-nixos that referenced this pull request Sep 28, 2018
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