Skip to content
This repository has been archived by the owner on Jun 24, 2021. It is now read-only.

Misc feature + fix package. #227

Open
wants to merge 4 commits into
base: release/4
Choose a base branch
from
Open

Misc feature + fix package. #227

wants to merge 4 commits into from

Conversation

jevolk
Copy link
Contributor

@jevolk jevolk commented Dec 5, 2016

  • Feature: Option to match oper confs based on vhost.

    • Looks finished; includes the security warnings.
  • Feature: SAJOIN command to force users to join channels.

    • Minimalistic, but might be enough for now.
    • "Hard" sajoins might not be possible without significant work to abstract m_join code.
  • Fix: This diff was necessary for me to build on a Windows10/msys2 platform.

    • I'm not really sure why, actually, and maybe this should be looked into before merging...

@jevolk jevolk changed the title B feature + fix package. Misc feature + fix package. Dec 5, 2016
use_id(client),
chname);

parse(client, buf, buf + len);
Copy link
Contributor Author

@jevolk jevolk Dec 6, 2016

Choose a reason for hiding this comment

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

Note: Check for local vs. remote client. ENCAP SAJOIN for remotes.

Copy link
Contributor

Choose a reason for hiding this comment

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

the parv comment is odd there

@Casey952
Copy link

For the Add flag to relax spoof restriction on the oper conf's hostmask, can the write up on reference.conf be added for ircd.conf aswell?

@Casey952
Copy link

for the use case having a staff vhost for the user address from which you can /oper with on the staff vhost like -- staff.network

@Mikaela
Copy link
Contributor

Mikaela commented Aug 6, 2018

Is there an intention to fix the merge conflicts and get this merged?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants