~samwhited/xmpp

xmpp: always start a server goroutine

Previously it was up to the user to start the server goroutine and if
they did not methods that rely on it such as SendIQ would block forever.
This patch changes this by always setting a server with a default
handler and allowing the user to update the handler (and block until the
server goroutine finishes) with a call to `Serve'.

The behavior of multiple `Serve' calls is still undefined.

Signed-off-by: Sam Whited <sam@samwhited.com>
all: update CHANGELOG.md with styling changes

Signed-off-by: Sam Whited <sam@samwhited.com>
all: update CONTRIBUTORS

Signed-off-by: Sam Whited <sam@samwhited.com>
1b17580e — Dane David 3 months ago
styling: use an iterator pattern for looping

Previously the Token method returned a token value and an error value,
so looping over a message styling document required a lot of extra
handling. To gain better usability, we have switched to an iterator
pattern by adding a Next method & altering the Token method.

Fixes #128

Signed-off-by: Dane David <dndavid102@gmail.com>
.builds: check author during DCO validation

Previously we were checking the committer which will be different if CI
is cherry picking the patch over or a different author is committing the
patch set.

Signed-off-by: Sam Whited <sam@samwhited.com>
.builds: don't run gosec on examples

Signed-off-by: Sam Whited <sam@samwhited.com>
xmpp: simplify TeeConn logic

This removes an ineffectual assignment where we set the reader or writer
to nil if the context timed out, but the nil check was useless because
the context would always be done and either branch would have returned
the same results.

Signed-off-by: Sam Whited <sam@samwhited.com>
jid: update fuzz tests to use new beta tests

These are known to be broken right now, but I believe it's an issue with
the Go branch and not with our tests. It will be investigated
eventually.

Signed-off-by: Sam Whited <sam@samwhited.com>
all: always build CONTRIBUTORS file

Because there are no file dependencies that can be checked to see if the
CONTRIBUTORS file is up to date it would never be regenerated. Instead,
make sure it is always regenerated.

Signed-off-by: Sam Whited <sam@samwhited.com>
styling: update fuzz tests to use new beta tests

Signed-off-by: Sam Whited <sam@samwhited.com>
styling: fix pre block starts with no newline

In a document that starts with three backticks and has no newlines we
still expect this to be the start of a pre-block.

Signed-off-by: Sam Whited <sam@samwhited.com>
all: update CONTRIBUTORS

Signed-off-by: Sam Whited <sam@samwhited.com>
43ad3cf4 — cemturker 3 months ago
xmpp: solve race condition in tests

Fixes #133

Signed-off-by: cemturker <cem.turker@messagebird.com>
examples/commands: fix button display

Sometimes a submit button would be missing from a form if no actions
existed, or a cancel button would be shown even if it was the only
action and can just be taken.

Signed-off-by: Sam Whited <sam@samwhited.com>
examples/commands: remove debug logging

Oops, this is why you should never be lazy and do printf debugging, just
use a debugger no matter how fast the language compiles. At least this
was just an example. Lesson learned (I hope).

Signed-off-by: Sam Whited <sam@samwhited.com>
commands: fix multi-stage commands

Signed-off-by: Sam Whited <sam@samwhited.com>
stanza: fix error unmarshaling with other payloads

Previously if the error was not the first payload in the stanza it would
not be found by UnmarshalIQError. Now we iterate through all the child
payloads looking for it (and disregarding other payloads) until we can
unmarshal it.

Signed-off-by: Sam Whited <sam@samwhited.com>
all: update main branch references in links and CI

Signed-off-by: Sam Whited <sam@samwhited.com>
.builds: use gosec v2

Signed-off-by: Sam Whited <sam@samwhited.com>
internal/integration: remove some gosec warnings

Newer versions of gosec warn about file inclsion from a variable, but
this is in integration tests in a safe manner so disable these warnings.

Signed-off-by: Sam Whited <sam@samwhited.com>
Next