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.

Sign up for GitHub

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

Jump to bottom

Move IRC to Libera.chat #291

Merged
merged 4 commits into from
Jul 21, 2021
Merged

Move IRC to Libera.chat #291

merged 4 commits into from
Jul 21, 2021

Conversation

teclator
Copy link
Contributor

It was agreed to move the our #YaST channel from Freenode to Libera.Chat, so this PR is for reflecting the change.

@teclator teclator requested a review from lslezak June 15, 2021 13:15
lslezak
lslezak reviewed Jun 15, 2021
View reviewed changes
contributing.html Outdated Show resolved Hide resolved
contact.md Show resolved Hide resolved
doc/contributing.md Outdated Show resolved Hide resolved
@crabbedhaloablution
Copy link

Pointing at irc.opensuse.org will break TLS verification, IIUC?

@teclator
Copy link
Contributor Author

IIUC

You are right

@crabbedhaloablution
Copy link

I mean, IRC isn’t the most secure protocol anyway, but I think it should be stated clearly before this is committed that breaking verifiable encryption is considered a bad move and tends to make a lot of people very angry.

@teclator
Copy link
Contributor Author

teclator commented Jun 16, 2021 •
edited
Loading

I mean, IRC isn’t the most secure protocol anyway, but I think it should be stated clearly before this is committed that breaking verifiable encryption is considered a bad move and tends to make a lot of people very angry.

Yep, I agree and in fact it is currently not mentioned anywhere, so I guess most of the people is using irc.opensuse.org without encryption (maybe without verifying the connection security) or them are using chat.freenode.net directly.

@lslezak lslezak mentioned this pull request Jul 16, 2021
Merged
ancorgs
ancorgs approved these changes Jul 21, 2021
View reviewed changes
Copy link
Contributor

@ancorgs ancorgs left a comment

Choose a reason for hiding this comment

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

LGTM

@teclator teclator merged commit 4f2c414 into master Jul 21, 2021
@teclator teclator deleted the move_to_libera branch July 21, 2021 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lslezak lslezak lslezak left review comments

@ancorgs ancorgs ancorgs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@teclator @crabbedhaloablution @lslezak @ancorgs