| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
| |
All errors were discarded to /dev/null if the error_ptr was set to
NULL.
|
|\ |
|
| |\
| | |
| | | |
Rework logging
|
| | |
| | |
| | |
| | |
| | |
| | | |
Changed default logging from INFO to WARN.
Issue: #1353
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Plain chat messages not working for non-carbon + no OTR support.
On master we did some clean-up. The problem is at https://github.com/profanity-im/profanity/blob/0.9.patch/src/event/server_events.c#L625 (0.9.0). The implementation looks like:
- HAVE_LIBOTR is set - _sv_ev_incoming_otr
- HAVE_LIBOTR is not set - _sv_ev_incoming_plain
I think the `_sv_ev_incoming_otr` can handle otr and plain, because I didn't find a `_sv_ev_incoming_plain` if `HAVE_LIBOTR` is set.
On master for 0.10.0 the implementation is much better:
https://github.com/profanity-im/profanity/blob/master/src/event/server_events.c#L623
But, we just call `_sv_ev_incoming_otr` independent of `HAVE_LIBOTR`.
Unfortunately, `_sv_ev_incoming_otr` is doing nothing if `HAVE_LIBOTR` is not set:
https://github.com/profanity-im/profanity/blob/master/src/event/server_events.c#L538
I did some more clean-up at sv_ev_incoming_message and changed the implementation of `_sv_ev_incoming_otr`.
```
static void
_sv_ev_incoming_otr(ProfChatWin *chatwin, gboolean new_win, ProfMessage *message)
{
// OTR or plain
plain
}
```
The caller do not take care of `HAVE_LIBOTR`, call `_sv_ev_incoming_plain` if you are sure it's a plain message or call `_sv_ev_incoming_otr`. `_sv_ev_incoming_otr` can be used for otr / plain or for plain only.
|
|\ \ |
|
| |\ \
| | | |
| | | | |
Request Device and Key, when OMEMO is in use
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Profanity requests the device list and keys for contacts, also when the user
hasn't generated the OMEMO key. If the user has no OMEMO key, there is no need
to request OMEMO information.
Issue: #1332
|
|\ \ \ \
| |/ / /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
In 0.9.x we fixed an issue, because OMEMO devices should be defined in "item"
with id "current". This should work, but it won't work if there is no "current".
If there is no "current" we will just use the first item.
Issue #1384
|
| | | |
| | | |
| | | |
| | | | |
This one will always be set.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
So that we don't have to pass the wrapping stanza and can handle
the error nicer.
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Trying to simplify the conditions so we don't have duplicate code
in both of those functions.
|
| | | |
| | | |
| | | |
| | | | |
Since its done in both cases.
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Authored by DebXWoody in:
https://github.com/profanity-im/profanity/pull/1369
Regards: https://github.com/profanity-im/profanity/issues/1366
Since I'm in the process of cleaning up message.c I take this now
so he doesn't have to rebase.
I also omitted the _handle_normal() case since I'm not sure that would
be correct.
Probably will be addressed again when continuing the cleanup.
|
| | | | |
|
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | | |
Messages from Conversations contains:
<request xmlns='urn:xmpp:receipts'/>
And would not be displayed in Profanity as it never reached
_handle_chat(..).
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
So far we logged when we receive a message without a type. Which is
actually quite common and makes no sense.
|
| | |
| | |
| | |
| | |
| | | |
RFC 6121 allows only few types.
So we can also remove that check in _handle_chat().
|
| | |
| | |
| | |
| | | |
Both cases are tested before entering that function.
|
| | |
| | |
| | |
| | |
| | | |
AFAIK it can only be one.
Except at STANZA_NS_MUC_USER which is used in several cases.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
It just does a free.
Related to b580b9ef119045f142fa4baa9689a1c5ce8864ef
|
|\ \ \
| |/ /
|/| | |
Rework /url and /executable for filetypes
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Additionally to:
7de83217c4d7bfdb70b77f2baeb8f12d4147d873
1e2a288d80fe0200f1d44d5106f7cc5bfd77718b
|
| | |
| | |
| | |
| | | |
not found
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
So far:
```
/executable urlsave html "test %u" results in url.save.cmd[html]=test %u
/executable urlsave * "test %u" results in nothing.
```
Probably due to limitation in .ini file format.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Additionally to ec7e635e752bdfef851fd177596c1d73d97afb42.
In the earlier commit I just setted the test value ignoring the real
format.
Now we correctly transform:
```
[logging]
urlopen.cmd=xdg-open
```
into:
```
[executables]
url.open.cmd=false;xdg-open %u;
```
|
| | | |
|
| | |
| | |
| | |
| | | |
Signed-off-by: Pierre Mazière <pierre.maziere@gmx.com>
|
| | |
| | |
| | |
| | | |
Signed-off-by: Pierre Mazière <pierre.maziere@gmx.com>
|