Write forceserver and webircpassword to conf

Make force local conn server configurable

Move webirc password to config

Fix build on modern libssl versions

Revert "Revert "Enable debugging by default""

This reverts commit 312c3ad63157e22eb37997eec7425da9accd9315.

Write config files less often with chansaver

Revert "Enable debugging by default"

This reverts commit fcf54795632f1d1f31a84e31587065ecf656f5d1.

Annotate detach message with number of clients remaining, if applicable.

This resolves the issue where users would be marked inactive if a detach occurs while another client is still attached.

Don't redefine _FORTIFY_SOURCE if compiler already defines it

autogen.sh: Check for pkg-config before calling automake

Signed-off-by: Uli Schlachter <redacted>
Conflicts:
autogen.sh

Don't make backup of znc.conf readable by everyone.

It doesn't really matter, since the whole ~/.znc/ is not readable, but
it's better to be safe.

Makefile: Remove unused (and broken) variable

Signed-off-by: Uli Schlachter <redacted>

route_replies: Add some WHOIS numerics

Charybdis can return all these.

Conflicts:
modules/route_replies.cpp

Make /znc detach case insensitive

Closes #213

Conflicts:
ClientCommand.cpp

DETACH should support the same syntax as PART (except no message)

Closes #225, this isn't the same as #225, but `/msg *status detach` is
something different, since it supports wildcard matching.

CModule::OnMode(): Fix a stupid NULL pointer dereference

When joining a channel, OnMode() (via SetModes()) was called with pOpNick ==
NULL. This bad pointer was turned into a reference and given to modules.

This bug exists since 2008 when the OnMode() module call was added. It wasn't
noticed before because apparently no module used this CNick argument before.

Signed-off-by: Uli Schlachter <redacted>
Conflicts:
Chan.cpp

Don't send our password required notice until after CAP negotiation

Conflicts:
Client.cpp

rfc1459 states that PART can take multiple channels

Parameters: <channel>{,<channel>}

Fixes #175

CString("off").ToBool() shouldn't be true

Fix Makefiles so we actually use RIZON, and clean up properly.

Remove module we don't use anymore.

Add Rizon flag to configure.ac, move Rizon modules into separate directory.

Don't print debug info to stdout with a debug compile if it is not isatty()

Fix DelClients(), removing clients can modify the client vector

Enable debugging by default

Also remove kicked users channels

Fix chansaver to not rewrite the config each time a user joins a channel on startup

Move away and notify_connect out of extras

Merge commit 'znc-0.206' into rizon

Why was this removed

Import of Rizon's ZNC-0.200

Increase the version number to 0.206

Signed-off-by: Uli Schlachter <redacted>

imapauth: Follow RFC more closely

The IMAP RFC allows the server to announce its capabilities before replying to
the LOGIN command. imapauth would misinterpret that as a failed login.

The fix is to only handle lines which contain the tag ("AUTH") used for the
login command.

Thanks to rlpowell for reporting that imapauth doesn't work against
imap.google.com and for testing the fix.

Signed-off-by: Uli Schlachter <redacted>

ResetJoinTries once we enable a channel

Fixes #83

Ask for port > 1024 in --makeconf

There're so many people who try lower ports and get permission denied :(
Who really wants to use privileged port, can add it using /znc addport

"no" should be false in CString::ToBool()

Make certs valid for 10 years

Previously, a certificate generated with --makepem expired after a year which is
weird and pointless (come on, it's self-signed!).

Signed-off-by: Uli Schlachter <redacted>

identfile: Make sure member values are NULL

Add Show command to identfile.

It should help us to debug that annoying bug of identfile aborting
connections...

CFile: Make sure errno is always set correctly

This fixes weird problems were strerror() made us print garbage.

Signed-off-by: Uli Schlachter <redacted>

CTCP's with no values should not be passed to the client

Fixes #105

Identfile: don't crash when ZNC is shutting down.

Thanks to Domin for reporting this.

Conflicts:

modules/identfile.cpp

Increase the version number to 0.204

Signed-off-by: Uli Schlachter <redacted>

autoreply: Honor RFC 2812.

There must not be autoreply for notices.

Thanks to nyuszika7h for noticing this.

Remove notes on the GENERATE command in the cert module

route_replies: Handle raw 482

lahwran reported the following message from *route_replies and also figured out
which message we failed to handle, thanks!

<*route_replies> This module hit a timeout which is possibly a bug.
<*route_replies> To disable this message, do "/msg *route_replies silent yes"
<*route_replies> Last request: MODE #somesecretchannel I

Signed-off-by: Uli Schlachter <redacted>

Change the order of -I directives in Makefile.

If ZNC was already installed, it had its headers somewhere.
The chances are that something else can be installed at the same place,
including some ZNC's dependency whose include dir would be included to
CXXFLAGS. Another possibility of including that dir is triggered when
using FreeBSD - ./configure explicitly adds -I/usr/local/include in that
case.

And so we get a directory with old ZNC headers included to CXXFLAGS
before our new shiny ./include.

With their order changed, the proper headers are included now.

Thanks to those who repored the issue, thanks to PsWii60 for helping to
track it down, thanks to my parents for creating me and therefore
enabling me to fix it, and to many other people.

Conflicts:

Makefile.in
modules/Makefile.in

Don't use "mkdir" during install

lahwran just showed up on irc and told us that he installed znc, but znc failed
to find any modules. The reason for this was his umask 077 which means that
"make install" installed stuff so that only root can access it.

The solution is do use "install -d" since that makes sure to ignore the
currently set umask.

However, google finds results which say that "install -d" might mess with stuff
of pre-existing directories when it shouldn't, so we must first test if the
directory already exists before calling install. Obviously, this makes our
Makefile a lot more readable. :-(

I didn't have time to test this properly, so stuff might break.

Signed-off-by: Uli Schlachter <redacted>
Conflicts:

modules/Makefile.in

Check for /usr/bin/swig2.0 too.

SWIG 2 is named that way on Debian-based systems...

identfile: Improve the debug messages

The new debug messages allow us to see when the identfile is unlocked

Conflicts:

modules/identfile.cpp

Improve admin module description.

Fscking perl... This fixes modperl's compilation.

This avoids your irc windows being filled with away stuff

Downside of antiidle is that because it messages yourself internally
the irc server will send messages to you when you're marked as away.
This can end up being really annoying, so those messages are blocked
with these changes.

Signed-off-by: Toon Schoenmakers <redacted>

Send a MODE request when JOINing

This kind of reverts/fixes f1cb09bd9ce0489d96f4ef91b3e226e0b0a37662.

Robby found a bug with evil ChanServ which goes like this:

- You join an empty, but registered channel and the IRCd applies its default
  modes (+nt), but ZNC didn't ask for MODE, so doesn't know this.
- ChanServ applies whatever channel modes it wants to apply. This causes ZNC to
  see a mode change (e.g. +s).

The result of this is that ZNC thinks that the channel has mode +s and it will
tell every client that connects to it about this.

The fix is to send a MODE request when JOINing. To make sure that we don't
confuse clients, we block the reply.

Signed-off-by: Uli Schlachter <redacted>

SHA256.cpp: Include some ZNC header first

This is required to make sure that zncconfig.h is the very first header that is
being included.

This problem was noticed by someone reporting some "macro redefined" warnings.

Signed-off-by: Uli Schlachter <redacted>

Move autocycle module to extra.

Modern/decent networks have services anyway, and for registered channels
this module is useless. Also it tends to fight ChanServ in case if
ChanServ joins/parts the channel to remove ops.

Users of deprecated ne^W^Wnetworks without services can use it from extra.

Drop @DEFS@ from the build system

This will only ever be set to -DHAVE_CONFIG_H. However, we shouldn't give this
flag to other people's code (e.g. through znc-config). Since we don't need it,
it's best to just drop it completely.

Signed-off-by: Uli Schlachter <redacted>

Add a message sent to user in case of redirect.

Let a user filter the result of help CModule::HandleHelpCommand

Don't forward server redirects to client.

Don't send server redirects to client [TorrentialStorm]

Unload modules in reverse order.

Increase the version number to 0.203

Signed-off-by: Uli Schlachter <redacted>

Fix crash in bouncedcc module.

It happens when DCC RESUME is received.
Affected ZNC versions: 0.200, 0.202.

Thanks to howeyc for reporting this and providing the patch.

Increase the version number to 0.202

Signed-off-by: Uli Schlachter <redacted>

fix formatting

rework SendBuffer fixing an issue where ClearBuffer() nuked the buffer after playing to the first client
(cherry picked from commit 793c34699a0de8a384cea359821beea66c1a00a4)

Conflicts:

Chan.cpp

Install .py files along with .pyc
(cherry picked from commit 5e73af632ab1f6e6112fdd983c9c8866fe1a0a9d)

fixed a null reference to pClient in hooks 'OnChanBufferStarting,OnChanBufferStarting,OnChanBufferEnding,OnChanBufferEnding', such that when pClient is NULL the hooks are called on all clients associated to that user

Conflicts:

Chan.cpp

Signed-off-by: Uli Schlachter <redacted>

Don't timeout bouncedcc connections, inactive bounced DCC chat's would be dropped otherwise

admin: Fix getchan {inconfig,keepbuffer,detached}

Instead of giving a useful result, they returned their result by sometimes
omitting the first character of the option name. Whoops.

Signed-off-by: Uli Schlachter <redacted>

Add -r parameter of ZNC to man.

It was missing.

Thanks to flaccid for noticing that.

fix build error when compiling against uclibc(++)

Fixes the following error:

In file included from ZNCString.cpp:10:0:
FileUtils.h: In static member function 'static CString CDir::GetCWD()':
FileUtils.h:246:36: error: 'getcwd' was not declared in this scope

Signed-off-by: Jonas Gorski <redacted>
Signed-off-by: Uli Schlachter <redacted>

autocycle: Only cycle once in 15 seconds

This should stop all fights against ChanServ. Please note that nothing will
happen if we are the only one in the channel after 15 secs since this module
only checks if it needs to do something when someone leaves a channel.

Signed-off-by: Uli Schlachter <redacted>

route_replies: Improve the timeout message

Fixes #57.

Signed-off-by: Uli Schlachter <redacted>

Fix CChan::TrimBuffer() (#63)

CChan::TrimBuffer() was trying to do "m_vsBuffer.begin() + -<some number>" which would cause a segmentation fault

Fixes #63

Increase the version number to 0.201

Signed-off-by: Uli Schlachter <redacted>

Increase the version number to 0.200

Signed-off-by: Uli Schlachter <redacted>

[cert module] Allow \r\n to be inside a certificate when uploading with the web interface.

Thanks to Robby for reporting

Fix uploading a certificate in the web interface for the cert module

Fix modp{erl,ython} installation

Signed-off-by: Uli Schlachter <redacted>

Fix module data for extra/ modules for real

Fix CString::Escape_n() and add some tests for it

This bug was originally reported by someone on irc, but sadly I forgot who it
was. Sorry!

Signed-off-by: Uli Schlachter <redacted>

Fix module data for extra/ modules

No module in extra/ comes with any data currently, so this was tested by moving
perform to extra/. The install is untested.

Signed-off-by: Uli Schlachter <redacted>

Include zncconfig.h in all headers

The rule is that zncconfig.h must be the very same thing that is included. We
cheat and only include it in headers (so that modules dont have to have be
changed). However, it looks like some modules where missed.

This commit fixes test/ConfigTest which crashed if _GLIBCXX_DEBUG was enabled
(--enable-debug) because it didn't see this define before including a c++
header.

Signed-off-by: Uli Schlachter <redacted>

Stop asking for the host name in --makepem

Come on, it's a self-signed certificate, how much value does it have anyway?
Also, it gets rid of a question during --makeconf which confuses some people.

This should now use $HOSTNAME and fall back to "host.unknown" if that isn't set.

Signed-off-by: Uli Schlachter <redacted>

Make the CUser::IsIRCConnected method check if the IRC connection is authed because may of its callers expect this behaviour.

This fixes a bug reported by Robby where a channel could become disabled while a the user is connected to irc, but before they have authenticated.

Fix a memleak in CChan::AddNick()

This memleak existed since 5e070e788 from almost a year ago.

Thanks to Olipro for reporting this.

Signed-off-by: Uli Schlachter <redacted>

Don't send a MODE request when JOINing

Since commit 1a1cc4c756e, we'd always send a "MODE #chan" to IRC when we
received a "JOIN #chan". It used to work before that commit, because the mode
reply which is automatically sent on join is sent before the "end of /names"
numeric.

This just removes that MODE request. Proper IRC server should always send a MODE
reply on channel join.

Thanks to nyuszika7h`` for the report.

Signed-off-by: Uli Schlachter <redacted>

Broadcast an error on EMFILE

When we hit the "maximum open files" limit, we close the listener that hit this
error. Previously we did so silently which means no one could figure out what
happened.

This commits adds a broadcast message giving a hint, but this should never
happen in a real-world situation anyway (unless you have 1000 users).

Signed-off-by: Uli Schlachter <redacted>

Fix verifying listener in --makeconf.

If ZNC is compiled with IPv6, ZNC tried to verify the listener using
IPv6 regardless of what user said.

Thanks to wodim for reporting this.

Hide every symbol except ZNCModInfo from modules.

This means that now it's possible to have the same class names in
different modules, which reduces chances of weird errors.

Thanks to mclc for noticing this.

Fix znc-buildmod if znc was compiled out of tree.

Update to latest Csocket

This fixes the "busy loop waiting for an SSL handshake to finish" which the last
"Update to latest Csocket" was already supposed to fix. However, that fix had a
bug if poll() is used instead of select().

poll()'s timeout argument is in milliseconds while select also allows
microseconds. Since Csocket originally used select(), it expects the
microseconds-approach. This means it has to divide by 1000 to get the timeout
argument for poll().

However, the iQuickReset which was used to "fix" (rather: hide) the busy loop
was less than 1ms so this still resulted in a timeout of 0 (= busy loop) because
integer division truncates the result.

Signed-off-by: Uli Schlachter <redacted>

WebMods: Set another cookie if the IP check fails

This means that they will get a new cookie on the next request. This will make
sure that you will be able to use webadmin again if your IP changed (which seems
to happen to quite a number of people).

Signed-off-by: Uli Schlachter <redacted>

znc-buildmod: Print znc's version number

This helps people figuring out that they are still building their modules for
the wrong znc version since they have more than one installation on their box.

Signed-off-by: Uli Schlachter <redacted>

Don't fail to load if can't find fixfreenode.

Remove fixfreenode module.

Old CAPAB method doesn't work with freenode now, and new CAP method is
handled by CAP support in the core.

Update to latest Csocket

Fixes:

- A possible crash bug for empty DNS replies with c-ares. E.g. a AAAA lookup for
  google.com doesn't give any reply but is still successful. This might be a
  c-ares bug (there is ARES_ENODATA) or c-ares just changed its behavior?
  (No bug report, just noticed accidentally)
- Connecting to ipv4-only hosts with a v6 bindhost caused weird errors:
  https://github.com/znc/znc/issues/47
- There was a pull request for some DSA server certificate thingy:
  https://github.com/znc/znc/pull/46
- Busy loop waiting for an SSL handshake to finish:
  http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=631590
- Some other stuff? No idea what some of the changes in here are actually doing.

Signed-off-by: Uli Schlachter <redacted>

Fix a bug in MCString::Encode()

For character values above 127, the signed char that could be used here did the
wrong thing. That is, *it >> 4 returned a negative value and the array hexdigits
was read indexed with that wrong value.

Fix this by explicitly using unsigned char.

Thanks to crocket for reporting this bug which he found with perform (broken
entries after a restart/reload).

Signed-off-by: Uli Schlachter <redacted>