about summary refs log tree commit diff
path: root/chat.h (unfollow)
Commit message (Collapse)Author
2021-06-09Replace catf with seprintfJune McEnroe
2021-06-09Add seprintfJune McEnroe
Based on seprint(2) from Plan 9. I'm not sure if my return value exactly matches Plan 9's in the case of truncation. seprint(2) is described only as returning a pointer to the terminating '\0', but if it does so even in the case of truncation, it is awkward for the caller to detect. This implementation returns end in the truncation case, so that (ptr == end) indicates truncation.
2021-06-09OpenBSD: pledge minimum promises from the startKlemens Nanni
catgirl needs: - "stdio tty" at all times - "rpath inet dns" once at startup for terminfo(5) and ssl(8) - "proc exec" iff -R/restrict options is disabled - "rpath wpath cpath" iff -s/save or -l/log options is enabled Status quo: catgirl starts with the superset of all possible promises "stdio rpath wpath cpath inet dns tty proc exec", drops offline with "stdio rpath wpath cpath tty proc exec" and possibly drops to either of "stdio rpath wpath cpath tty", "stdio tty proc exec" or "stdio tty" depending on the options used. Such step-by-step reduction is straight forward and easy to model along the process runtime, but it comes with the drawback of starting with too broad promises right from the beginning, i.e. `catgirl -R -h host' is able to execute code and write to filesystems even though it must never do so according the (un)used options. Lay out required promises up front and pledge in two stages: 1. initial setup, i.e. fixed "stdio tty" plus temporary "rpath inet dns" plus potential "rpath wpath cpath" plus potential "proc exec" 2. final rutime, i.e. fixed "stdio tty" plus potential "rpath wpath cpath" plus potential "proc exec" This way the above mentioned usage example can never execute or write files, hence less potential for bugs and more accurate modelling of catgirl's runtime -- dropping "inet dns" alone in between also becomes obsolete with this approach.
2021-06-09OpenBSD: unveil after ncurses(3) init to support TERMINFOKlemens Nanni
initscr(3) in uiInitEarly() attempts more than /usr/share/terminfo/, see `mandoc -O tag=TERMINFO ncurses`. Even though non-default terminfo handling seems rare and it is unlikely to have ever caused a problem for catgirl users on OpenBSD, the current is still wrong by oversimplifying it. Avoid the entire curses/unveil clash by setting up the screen before unveiling.
2021-06-09OpenBSD: hoist -o/printCert code to simplifyKlemens Nanni
Nothing but the TLS handshake is required, so skip all other setup. On OpenBSD, unveil() handling needs fixing which will involve code reshuffling -- this is the first related but standalone step. Also pledge this one-off code path individually such with simpler and tighter promises while here.
2021-06-06Pad kiosk username with zero, not spaceJune McEnroe
Oops!
2021-06-06OpenBSD: unveil XDG directories only when neededKlemens Nanni
The (not perfectly obvious) way catgirl crafts directories gets triggered by unveilAll() even if no passed option requires filesystem access: $ env -i TERM=xterm ./catgirl -h irc.hackint.eu -R -n nobody catgirl: HOME unset Here unveil(2) is used due to the "restrict" option, but besides terminfo(5) and certificates catgirl does not need any other files, yet it tries to init the data path -- passing XDG_DATA_HOME=/var/empty makes above invocation work showing how the then successful path setup is not required. Fix this by not unveiling the unneeded data path in the first place.
2021-06-05Nickname defaults to system's username not IRC usernameKlemens Nanni
"username" alone is ambiguous and without jumping to ENVIRONMENT explaining the use of USER, catgirl's user- and nickname options read like pointing at each other: -n nick | nick = nick Set nickname to nick. The default nickname is the user's name. [...] -u user | user = user Set username to user. The default username is the same as the nickname. Clarify that `-n' does *not* default to `-u's value.
2021-06-05Avoid writing past the end of the status barMichael Forney
When waddnstr is called with a string that would extend past the end of the window, the string is truncated, the cursor remains at the last column, and ERR is returned. If this error is ignored and the loop continues, the next call to waddnstr overwrites the character at this column, resulting in a slight visual artifact. When the window is too small to fit the full status line, it is effectively truncated by one space on the right, since the string shown for each channel begins with a space. Additionally, if the last window is the current window, the space is shown with a colored background. To fix this, when waddnstr returns ERR, exit the loop in styleAdd() early return -1 to propogate this error down to the caller.
2021-05-28List windows with /window 1.8June McEnroe
Reuse the /window command to preserve /wi abbreviation.
2021-05-28Improve missing param behavior for /msg, /whois, /ns, /csJune McEnroe
2021-05-28Use | for /window | /num commandJune McEnroe
2021-05-28Prefix = for options with No in manualJune McEnroe
Without, the mandoc HTML output includes the space and equals in the class="Cm" element and generates a permalink of #hash_= for example.
2021-05-27Tag config options in manualKlemens Nanni
E.g. ":t debug" will now jump right to the definition just like ":t v" already did -- at least with mandoc(1) from OpenBSD.
2021-05-27Document channel key parametersJune McEnroe
2021-05-27Set username from SSH_CLIENT in chrootJune McEnroe
2021-05-27Hash the username in kiosk modeJune McEnroe
So that the first part of $SSH_CLIENT can be passed as username.
2021-05-27Log nick and ssh connection in chroot-promptJune McEnroe
2021-05-25Clarify -H hash optionJune McEnroe
2021-05-25Use reverse video not colors for topic change when disabledKlemens Nanni
`-H 0,0`/"hash = 0,0" makes catgirl mostly colorless which is great, but topic changes still hardcode brown/green colors to show differences which is usually not desired by users (like me) disabling colors. Go for a less eye stressing topic change message that shows both old and new in reverse video with default terminal colors. This isn't perfect, other parts of catgirl still hardcode colors and `-H 0,0`/"hash = 0,0" was never meant to disable colors completely, but topics change often enough that avoiding less readable^Waccessible topic diffs seems sensible enough. NB: parseHash() is brittle and "0,0" is not the only value disabling colors...
2021-05-25Use color enum instead of hardcoded valueKlemens Nanni
2021-05-20Don't require 4 parameters to ERR_USERONCHANNELJune McEnroe
It should have 4, but the handler only uses 3.
2021-05-19Replace freenode with tilde.chatJune McEnroe
2021-05-04Remove no longer needed advice about mandirJune McEnroe
2021-05-04Ignore messages in reply to previously ignored messagesJune McEnroe
Using the +draft/reply client tag, which is supported by BitBot. This hides the bot's replies to ignored users or ignored bot command messages. This commit is dedicated to the land of Estonia.
2021-05-04Add support for BINDIR, fix default MANDIR, use LDADD varsJune McEnroe
I avoided defaulting MANDIR to /usr/local/man because I thought it didn't work on GNU/Linux and users would be confused, but it turns out man-db's default configuration includes both /usr/local/man and /usr/man, so ${PREFIX}/man is a sensical default.
2021-04-27Reset formatting after realnamesJune McEnroe
2021-04-16Set id color from completion in /msgJune McEnroe
In the same way that /query copies the id color from completion. Also make both first check that a color isn't already set.
2021-04-16Always show 341 RPL_INVITINGJune McEnroe
At least in InspIRCd's implementation, you only get invite-notify INVITEs if you are op, so inviting with no op (where allowed by a channel mode) results in only a 341. On the other hand, inviting as an op produces both a 341 and an INVITE, so will be displayed twice, but showing something sometimes twice is better than not showing it at all.
2021-04-02Skip STATUSMSG prefixesJune McEnroe
This feature is rarely used, so just skip STATUSMSG prefixes in the target so messages get routed correctly.
2021-03-18Fix README typo 1.7June McEnroe
How long has this been here?
2021-03-17Add C-z keys for directly inserting most color codesJune McEnroe
So you don't have to remember those dang numbers whose order makes no sense!
2021-03-17Reset style after newline in inputJune McEnroe
Reflect what will actually be sent.
2021-03-17Show where too-long-messages will be automatically splitJune McEnroe
2021-03-14Allow multi-line /me and split long /me messagesJune McEnroe
/me shouldn't behave differently from a regular message.
2021-03-13Refer to glob(7) not sh(1)Klemens Nanni
Those patterns are not specific to the shell, many commands support them. Notes: glob(7) does not exist on FreeBSD, but I'm going to consider that a documentation bug on FreeBSD's part. The page exists in OpenBSD, NetBSD and even Linux!
2021-03-13Add note about arrow and navigation keysJune McEnroe
And I think with C-Left and C-Right I can actually say "as expected" now.
2021-03-10chat.tmux.conf: use config_files format for reloadKlemens Nanni
The recent addition of "#{source_files}" allows us to avoid hardcoding the file name and instead ask tmux itself for the very file it used to create the session in the first place, i.e. "-f ./chat.tmux.conf".
2021-03-08Attempt to clarify trust option useJune McEnroe
Trust is not certificate pinning and should only be used for self-signed certificates.
2021-03-08Disable nick and channel colors with hash bound 0June McEnroe
2021-03-07Add bindings for C-Left and C-RightJune McEnroe
Apparently these are common. There's no terminfo for these, so manually define the xterm sequences. There's no documentation in the manual for the "intuitive" keys... I'm not sure if that should continue to be the case or not.
2021-03-02Add all window names to global completionJune McEnroe
Don't want to be touching window names much though, otherwise query window names would interfere with tab completion within a channel.
2021-02-27Add workaround for lack of A_ITALIC in old ncursesJune McEnroe
A_BLINK has probably always existed, but there's no good reason to ever use it, so make it do italics instead. Normally all attributes are set by a single set_attributes string if it's set, so clear it to force ncurses to use the reassigned enter_blink_mode string. If the terminal has no enter_italics_mode string, then nothing will happen. This makes setting multiple attributes a bit less efficient, but I don't think it's likely to make much of a difference since using multiple attributes at once is so uncommon.
2021-02-25Error if hash bound is less than 2June McEnroe
Bad things happen otherwise.
2021-02-21Use separate reply counts for automatic join topics/namesJune McEnroe
This restores showing the topic and names for automatic joined channels, while still avoiding touching the windows, by using Cold heat.
2021-02-15Add C-z p to manually toggle paste modeJune McEnroe
OpenBSD's xterm doesn't have bracketed paste mode, and it would be nice to still be able to paste in several lines and collapse them with M-q, provided one remembers to type C-z p first...
2021-02-15Refactor Focus and Paste mode stringsJune McEnroe
2021-02-15Don't insert wchar_t if it can't be converted to mbsJune McEnroe
Otherwise it could hit the assertion in editBuffer while converting to mbs for consumption by the rest of the program. It's possibly to trigger this with LC_ALL=C and typing C-z C-v M-a, for example.
2021-02-15Don't expand macro if it can't be converted to mbsJune McEnroe
Otherwise expanding a macro could hit the assertion in editBuffer while converting to mbs for consumption by the rest of the program.
2021-02-08Add donation link to README 1.6June McEnroe