aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
* skype: fix tests to expect NOTICE * instead of NOTICE AUTHdequis2015-10-13-21/+21
| | | | Forgot to change this after b39859e
* skype: use g_vsnprintf() instead of vsnprintf()dequis2015-10-13-2/+1
| | | | | | More prefixes = better. The G stands for "good". (it also fixes the warning about _BSD_SOURCE being deprecated)
* skype: use ssl_sockerr_again() instead of sockerr_again()dequis2015-10-13-1/+1
| | | | | Found the thing using 100% cpu because of a dead connection that it didn't want to bury.
* twitter: Fix some nitpicky issues reported by coveritydequis2015-10-08-21/+24
| | | | | | Mostly minor rare leaks that happen in error conditions, and one dereference before null check in twitter_logout (the null check is probably the wrong one there, but it doesn't hurt to keep it)
* irc_sasl_plain_parse: fix size of the parts arraydequis2015-10-08-1/+1
| | | | | | | | Oddly enough it doesn't break. Maybe it overwrites the 'decoded' pointer when it doesn't need it anymore? Fun stuff. Also the version of gcc (5.2) i'm using doesn't complain about array bounds, clang does, and the older gcc (4.6) in travis does too.
* Fix use-after-free when leaving invited (temporary) channelsdequis2015-10-08-2/+5
| | | | | | | | | Trac ticket 1229, https://bugs.bitlbee.org/bitlbee/ticket/1229 This one is complicated. See the trac ticket for details. Relevant commits: cc20520 6963230 664bac3 - second one fixed the same issue, third one reverted it, this one takes a different approach.
* The SASL PLAIN RFC says that the first part can be emptydequis2015-10-08-2/+2
| | | | | | | | | | | | | | | | | So use the second part as the username, and only require it to be equal to the first part if that one is present. ABNF from the spec: message = [authzid] UTF8NUL authcid UTF8NUL passwd Note brackets. Authzid (authorization identity) is meant for impersonation, which we don't support. The actual login username is defined by authcid (authentication identity) Thanks grawity for pointing this out.
* IRCv3 SASL capability + PLAIN methoddequis2015-10-08-0/+143
| | | | | | | | | | | | | | | | | | | | | Only plain, no other methods. We don't have built-in SSL to implement EXTERNAL (certfp) and nothing else is worth implementing. The actual authentication is pretty much like sending a server password (when the server's authmode isn't closed), which means it happens in cmd_identify, but this code also calls storage_check_pass() to send the required success/failure replies. SASL doesn't give us much benefit other than standards compliance, but some clients might appreciate it. And having a fifth way to do the same thing doesn't hurt! Now we have: - identify in &bitlbee - identify to nickserv (alias for root) - 'nickserv' and 'ns' irc commands - server password - sasl plain
* irc_send_who: move the iu->flags check after iu has a value...dequis2015-10-08-3/+3
| | | | | | Moving this was suggested during the review of the multi-prefix patch as ordering the sets of status_prefix... and it seemed reasonable to me. Thanks valgrind.
* IRCv3 multi-prefix... but mostly just adding prefixes to WHOdequis2015-10-08-17/+34
| | | | | | | | | We can't actually have multiple prefixes internally, so the only thing missing for multi-prefix compliance is actually having the prefix in the WHO reply, which is a rfc1459 thing. Note to future self: check irc logs for the implementation I threw away. The one that actually handled multiple prefixes. I hope that's useful.
* Allow NULL callback functions in http_dorequestArtem Savkov2015-10-08-3/+7
| | | | | | | | | Check callback function supplied to http_dorequest and only run it if it is not NULL. While it is not the usual case there are some times when there is no need to check the results of a http request. Using a NULL pointer is much more convenient than creating noop functions.
* Replace <br/> and <br /> with \n in strip_htmlAndré Glüpker2015-10-08-0/+4
|
* Add missing space in configureAndré Glüpker2015-10-08-4/+4
|
* Fix typoMohammed Attia2015-10-08-1/+1
|
* jabber: Fix leak in jabber_gmail_handle_newdequis2015-10-08-2/+2
| | | | | | | From coverity. That g_strdup_printf() was really pointless, slightly ashamed we didn't notice that in the review of the patch.
* conf, help: Fix minor leaks in error conditionsdequis2015-10-08-0/+5
| | | | From coverity.
* jabber: fix null handling with MUC topicsdequis2015-10-08-1/+2
| | | | From coverity.
* Move CAP commands to irc_cap.c; use enum for flagsdequis2015-09-30-151/+190
|
* CAP LISTdequis2015-09-10-1/+24
|
* CAP LSdequis2015-09-10-6/+18
|
* CAP REQdequis2015-09-10-2/+90
|
* Initial implementation of ircv3 capability negotiationdequis2015-09-10-1/+38
| | | | | Mostly no-op for now. Puts registration on hold, supports the basic commands, and NAKs everything
* Allow setting the plugin dir at runtimeanderspapitto2015-09-05-0/+3
| | | | | | | This enables the use of bitlbee plugins in scenarios where there is no write access to the bitlbee lib/ directory. One example is the NixOS linux distribution (which I'm currently packaging a bitlbee plugin for), where post-installation modification of a package (e.g. bitlbee) by another package (e.g. bitlbee-facebook) is not possible. Another example would be a user without root access building and using a plugin with a system-provided (i.e. installed by root) bitlbee.
* Improved cross compiler support.Guillermo A. Amaral2015-08-31-4/+27
| | | | | | | | | * Added a configure option for sysroot. * Prevent pkgconfig environment variables to be clobbered. * Prevent PATH environment variable pollution. * Backward compatible. Signed-off-by: Guillermo A. Amaral <g@maral.me>
* user-guide: add help for the jabber 'proxy' settingdequis2015-08-27-0/+20
|
* irc_channel_name_gen: handle g_convert_with_fallback failuresdequis2015-08-27-0/+10
| | | | | | | | | | First fallback to ASCII without TRANSLIT, and if that fails too, just give up by returning NULL. Basically the same thing as 3a27896 (a netbsd specific fix), but for channel names. This wasn't needed before because the older version of this code caught the NULL from the ASCII//TRANSLIT attempt and gave up immediately, while the refactored version lacked null checking.
* jabber: Improvements to the MUC part reason handlingdequis2015-08-26-15/+26
| | | | | | | - Look for a status message right inside <presence> (seen with ejabberd as a result of a s2s connection error) - Check status codes in a while loop, skipping unknown ones (such as 110, which means "Inform user that presence refers to itself")
* jabber/io: Split input buffer parsing to a jabber_feed_input() functiondequis2015-08-26-51/+71
| | | | | | | | | | | To simplify testing. Also allow passing a -1 as size to use strlen() Minor behavior change: The jabber_init_iq_auth() branch can no longer return immediately, which means it will continue through the ssl_pending() check in jabber_read_callback(). Other than that, the size -1 change, and one indentation level less, the function body is the same as before.
* Makefile: suggest users to run "make install-dev" after installdequis2015-08-26-2/+6
| | | | Also add a couple of missing targets to .PHONY
* conf: Fix leak of members of conf_t when using -c to specify a configdequis2015-08-19-1/+30
| | | | | | Can only happen once, and it's just ~200 bytes. But being valgrind-clean is good.
* irc: Replace NOTICE AUTH before registration with NOTICE *dequis2015-08-18-3/+3
| | | | | | Because many other ircds do that. (But actually mostly because @SaberUK said so)
* otr: add otr_filter_colors, replaces '\x03' with '?' for "security"dequis2015-08-11-2/+16
| | | | | | | | | | | | | Fixes trac ticket 835, "an attacker can spoof color codes" Which had "major" priority, and was open for a few years. Yeah. Every time I looked at that ticket I thought about lowering the priority, but then saw that pesco opened the bug. Welp. Anyway, it's gone now. Yay.
* otr: color multiline messagesdequis2015-08-11-24/+51
| | | | | | | | | | | Fixes trac ticket 710. Incoming messages can have newlines in them, which become several PRIVMSG on the irc layer. Prepending color codes at the beginning of the message resulted in showing the rest of those PRIVMSG as white. This splits the message by newlines and rebuilds it in a GString, re-adding the color codes right after each newline.
* jabber_logout: avoid null jd->buddies warnings in jabber_buddy_remove_alldequis2015-08-08-1/+3
|
* jabber: Show MUC kick/ban/leave reasonsdequis2015-08-08-2/+34
|
* Pass 'reason' values around in imcb_chat_remove_buddy/chat_remove_userdequis2015-08-08-4/+4
|
* travis: Switch to container based infrastructuredequis2015-08-07-5/+15
| | | | | This means faster builds, which is something I don't really need but I felt like doing this.
* configure: use pkg-config for libotrdequis2015-08-07-22/+14
|
* typofix - https://github.com/vlajos/misspell_fixerVeres Lajos2015-08-07-58/+58
|
* Merge remote-tracking branch 'origin/release-3.4.1'Wilmer van der Gaast2015-08-03-3/+5
|\
| * 3.4.1-1 Debian package.Wilmer van der Gaast2015-08-03-3/+5
| |
* | Only try DIGEST-MD5 if there's no SSL/TLS or if PLAIN isn't supporteddequis2015-07-30-1/+5
| | | | | | | | | | | | | | Which in practice means "don't bother with DIGEST-MD5 most of the time". It's weak, pointless over TLS, and often breaks with some servers (hi openfire)
* | Merge branch 'develop'dequis2015-07-30-4/+34
|\ \
| * | jabber: Add SASL ANONYMOUS support (XEP-0175)dequis2015-07-04-3/+33
| | | | | | | | | | | | Use "account jabber set anonymous on" to have bitlbee try that method
| * | jabber/s5bytestream: add G_GNUC_PRINTF to jabber_bs_abortdequis2015-06-28-1/+1
| | |
* | | Replace `backticks` with $(something better) in configure script.Wilmer van der Gaast2015-07-29-26/+26
| | |
* | | Restore old autojoin logic and a hopefully better fix for the problemWilmer van der Gaast2015-07-29-5/+6
|/ / | | | | | | jgeboski was trying to solve. #1221 for details.
* / Fix NULL pointer dereference in lib/json.c. (#1220)Wilmer van der Gaast2015-06-17-1/+1
|/
* Update changelog with date and other bits.3.4.1Wilmer van der Gaast2015-06-16-3/+3
|
* 3.4.1 release notesdequis2015-06-14-2/+36
|