about summary refs log tree commit diff stats
Commit message (Collapse)AuthorAgeFilesLines
...
| * Switch to clang-format check actionMichael Vetter2023-04-141-17/+6
|/ | | | | | | | | | | | Instead of running clang-format outselves on the old Ubuntu version. This let's us easily configure which version of clang-format we want to execute. Used action: https://github.com/marketplace/actions/clang-format-check Properly fix: https://github.com/profanity-im/profanity/pull/1774
* docs: Fix formatting in SECURITY.mdMichael Vetter2023-04-141-1/+2
|
* Merge pull request #1823 from H3rnand3zzz/fix/msg-crashMichael Vetter2023-04-142-7/+8
|\ | | | | Fix memory corruption crash
| * Cleanup p_ox_gpg_decryptJohn Hernandez2023-04-131-4/+6
| | | | | | | | | | | | In OX implementation gpgme's buffer remains untouched, thus not leading to the crash. But code can be shorter and more concise.
| * Fix memory corruption crashJohn Hernandez2023-04-131-3/+2
| | | | | | | | | | | | | | Under certain circumstances setting plain_str[len] to 0 might lead to crash and it does not follow the best practices as well. This change allows better handling of buffer copying and prevents crash.
* | Merge pull request #1826 from H3rnand3zzz/feature/roster-nickname-removeMichael Vetter2023-04-149-7/+65
|\ \ | |/ |/| Add nickname support for `/roster remove`
| * Add nickname support for /roster removeJohn Hernandez2023-04-139-7/+65
|/ | | | | | Add support of name/nickname instead of only JID for `/roster remove` command. Add tests for it as well.
* docs: add SECURITY.mdMichael Vetter2023-04-131-0/+6
|
* iq: initialize os and os_txtMichael Vetter2023-04-111-2/+2
| | | | | | | | | | | | Let's set this to NULL. Someone was using `picaur` and got an error about potential uninitialization. Even though the code is fine let's set this to NULL to make the compiler happy. The strange thing was that building manually on the same system worked. Related to 2e43b0ae62478bede34f6f52f73d22d671425f85.
* Merge pull request #1822 from H3rnand3zzz/fix/iq-osMichael Vetter2023-04-111-3/+2
|\ | | | | Hotfix stanza release
| * Fix releases of os and os_txt stanzasJohn Hernandez2023-04-101-3/+2
| | | | | | | | | | | | | | | | | | | | If custom client is not set and include_os is true, stanzas might be released without initializiting. This commit fixes it by introducing an additional check on usage of custom client. Related to commit e52ca2fbaa02acec3b2b757369ef5740e7a103da
* | Add test/valgrind section to PR templateMichael Vetter2023-04-101-0/+6
| |
* | Add limitation info about /statusbar chat user|jidMichael Vetter2023-04-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While testing https://github.com/profanity-im/profanity/pull/1817 we found that: ``` /statusbar show name (I have this off by default /msg someone where I chose someone in my roster that has a nick assigned /statusbar chat jid ``` Will only take effect upon creation or redraw of that tab. Reason is that we do this in create_tab to limit building this dynamically. Let's leave it like it is but add a note to users. Not worth build that string all the time.
* | Merge pull request #1817 from H3rnand3zzz/feature/full-jidMichael Vetter2023-04-103-25/+57
|\ \ | |/ |/| JID Display in Titlebar and Fix
| * Fixes Statusbar tabs and CleanupJohn Hernandez2023-04-102-15/+39
| | | | | | | | | | | | | | | | | | | | - Fixes statusbar tabs. Bug: Statusbar used nickname if it was set in roster, irrelevant to /statusbar chat setting. Expected behaviour would be using this setting set as "user" to show nickname, and to show jid with "jid" setting. Other solution is to give a user control over it with another settings. - _status_bar_draw_maintext cleaned up, no changes to behaviour
| * Improve titlebar flexibilityJohn Hernandez2023-04-102-11/+19
|/ | | | | | | Before this change, only nickname or JID (if no nickname set) is shown in the titlebar. Change allows to alter nickname/JID preference using already implemented /titlebar show jid|name setting. If both are set to "show", then it is displayed in "nickname <JID>" format.
* contributing: add note about testingMichael Vetter2023-04-091-0/+1
|
* Rewrite description for clientidMichael Vetter2023-04-091-1/+1
|
* Merge pull request #1815 from H3rnand3zzz/feature/the-client-switcherMichael Vetter2023-04-0916-86/+151
|\ | | | | Feature: Allow setting client identification name/version manually
| * Allow setting client identification name/version manuallyJohn Hernandez2023-04-0916-86/+151
| | | | | | | | | | | | | | | | Add changes allowing user to switch client name and version. Useful for enhancing user privacy. Minor cleanup.
* | Merge pull request #1821 from mdosch/use-proper-ellipsis-charMichael Vetter2023-04-0910-28/+28
|\ \ | | | | | | User proper ellipsis char
| * | User proper ellipsis charMartin Dosch2023-04-0910-28/+28
|/ / | | | | | | | | | | | | As stated in https://github.com/profanity-im/profanity/pull/1820#issuecomment-1498083383 profanity uses "..." (three dots) in a lot of places instead the proper ellipsis char "…".
* | Merge pull request #1813 from profanity-im/tabmode-irssiMichael Vetter2023-04-087-46/+135
|\ \ | | | | | | add `/statusbar tabmode actlist`
| * | add `/statusbar tabmode actlist`Steffen Jaeckel2023-04-087-46/+135
|/ / | | | | | | | | | | | | | | | | | | The existing way how active tabs are displayed didn't allow showing more than 10 tabs. This patch adds a mode where the statusbar shows a comma-separated list of tabs which were active since the last time viewed. This view is inspired by how `irssi` shows the active tabs, therefore it is also called `actlist`. Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
* | Merge pull request #1814 from profanity-im/improvementsMichael Vetter2023-04-043-113/+118
|\ \ | | | | | | Improvements and bugfixes
| * | fix display of a "list type" in a form if there's no value assigned yetSteffen Jaeckel2023-04-041-27/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is possible, that a server sends a form with a field as follows, which has no (default) value assigned. ``` <field label="foo" type="list-single" var="bar"> <option label="a"><value>a</value></option> <option label="b"><value>b</value></option> <option label="c"><value>c</value></option> </field> ``` This patch fixes profanity to show that list. Before this patch profanity showed nothing. I stumbled over this while running the `/room config` command inside a newly created muc on the `sure.im` XMPP server. Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
| * | refactor into array of a `struct`Steffen Jaeckel2023-04-041-5/+15
| | | | | | | | | | | | | | | | | | ...instead of having two separate arrays. Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
| * | only create hashtable onceSteffen Jaeckel2023-04-041-72/+74
| | | | | | | | | | | | | | | | | | | | | Before this change the same hashtable was re-created each time one used the auto-completion feature. Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
| * | fix memory leakSteffen Jaeckel2023-04-041-9/+8
|/ / | | | | | | | | | | | | | | `found` was leaked before This was somehow missed in e9aaba938b4f1b62b5f9018583a82ed7e7180977 Signed-off-by: Steffen Jaeckel <jaeckel-floss@eyet-services.de>
* | Merge pull request #1811 from shahab-vahedi/batman-fixMichael Vetter2023-04-031-2/+2
|\ \ | |/ |/| Fix color name in batman theme
| * themes/batman: Use "bold_black" iso "black_bold"Shahab Vahedi2023-04-031-2/+2
| | | | | | | | | | | | | | | | | | | | This makes all the related errors in the log files go away: ... prof: ERR: Color: bad color name black_bold_default prof: ERR: Unable to load colour theme ... Fixes https://github.com/profanity-im/profanity/issues/1810
* | Fix typosMichael Vetter2023-04-034-4/+4
| |
* | Merge pull request #1808 from ↵Michael Vetter2023-04-032-1/+5
|\ \ | | | | | | | | | | | | MarcoPolo-PasTonMolo/fix/empty_window_after_reconnect Fix empty window after reconnect
| * | Fix empty window after reconnectMarcoPolo-PasTonMolo2023-04-022-1/+5
| |/ | | | | | | | | | | | | | | Sometimes after a reconnect the current window would get cleared. This was a deliberate change to fix the profanity window looking all garbled up after providing the passphrase for a gpg key using pinentry-curses. Fixes https://github.com/profanity-im/profanity/issues/1556
* | Merge pull request #1809 from MarcoPolo-PasTonMolo/fix/no-ac-for-reconnect-nowMichael Vetter2023-04-031-0/+16
|\ \ | | | | | | Add autocomplete for the /reconnect command
| * | Add autocomplete for the /reconnect commandMarcoPolo-PasTonMolo2023-04-021-0/+16
| |/ | | | | | | | | After typing `/reconnect ` and pressing tab nothing appeared where in reality, it should have been autocompleted to `/reconnect now`
* | Merge pull request #1807 from alexandre1985/fix/typosMichael Vetter2023-04-034-5/+8
|\ \ | |/ |/| Fix typos and add codespell configuration
| * Fix typos and update codespell configurationDaniel Santos2023-04-034-5/+8
|/ | | | | | | | | | * Fix typos. * Add words that are not typos to codespell's ignore words list and ignore regex. * Make codespell ignore URIs. * Make `make doublecheck` throw no error. Signed-off-by: Daniel Santos <dacs.git@brilhante.top>
* Merge pull request #1801 from alexandre1985/editor-with-argumentsMichael Vetter2023-03-312-2/+8
|\ | | | | Editor executable support flags
| * Add editor executable to CMD_SYNDaniel Santos2023-03-311-0/+1
| | | | | | | | | | | | * Add missing text of how to set editor in CMD_SYN Signed-off-by: Daniel Santos <dacs.git@brilhante.top>
| * Editor executable support flagsDaniel Santos2023-03-302-2/+7
| | | | | | | | | | | | | | | | * Make editor executable into a string to be able to support (multiple) flags. * Change /help executable to suit this new feature Signed-off-by: Daniel Santos <dacs.git@brilhante.top>
* | Fix headings in contributing.mdMichael Vetter2023-03-311-2/+2
| |
* | Merge pull request #1802 from H3rnand3zzz/pgp-fixMichael Vetter2023-03-314-17/+10
|\ \ | |/ |/| Security fix of OTR, PGP and other
| * Fix OTRJohn Hernandez2023-03-311-4/+0
| | | | | | | | Fix OTR: carbon in stanza breaks OTR and prevents it from starting and working properly.
| * Fix PGPJohn Hernandez2023-03-313-13/+10
|/ | | | Fix of PGP and potentially other encryption methods by calling correct function that would initiate them in case if someone writes.
* Merge pull request #1800 from profanity-im/switchMichael Vetter2023-03-233-33/+39
|\ | | | | Improve some UI functions
| * Add WIN_VCARD to win_get_tab_identifier()Michael Vetter2023-03-231-0/+4
| | | | | | | | | | | | Thanks to 12b997c5f34776f34634d4ca155a14586aebc905 we already found a place where we should add this missing window type. AFAIK there can be only one vcard window.
| * Adapt to new g_string_free() usageMichael Vetter2023-03-232-13/+4
| | | | | | | | | | Somehow this must have been overlooked when doing e59c401c840f379e64945734969db03b0e55ef22.
| * ui: make it easier to find non covered window typesMichael Vetter2023-03-231-20/+31
|/ | | | | | | | | | | | | | | | Rewrite `win_get_title()` to using switch without a default case. So the compiler warns us (`enumeration value ‘WIN_XXX’ not handled in switch`) in case we add a new window type (WIN_CHAT, WIN_PRIV etc) and forget to adapt this function. Add assert() in the end to make compiler happy when he realizes there the function might have no return value (`control reaches end of non-void function`). This should ever be reached. Replace the default value of `win_to_string()`, `win_get_tab_identifier()` as well, and replace it with an assert. See discussion started at https://github.com/profanity-im/profanity/pull/1799#discussion_r1142444684
* Merge pull request #1799 from profanity-im/glib276gsfMichael Vetter2023-03-2217-145/+42
|\ | | | | Adapt to g_string_free glib 2.75.3 change