Emoji in nicks causes miscalculation of bytes in long message splitter #4

Closed
opened 5 months ago by foxiepaws ยท 3 comments
Owner

A miscalculation occurs when a display name has emoji in it.

foxie ๐ŸฆŠ ๐Ÿบ ๐Ÿฆฆ ๐Ÿ‰ ๐Ÿˆโฌ› ๐Ÿ† ๐Ÿฆจ ๐Ÿฆ โ€” Today at 11:53 PM
Pellentesqueluctustinciduntarcu,idportanequevolutpatut.Etiammagnafelis,pulvinaraerosin,molestieposuerearcu.Quisqueconsecteturnuncnonposueretincidunt.Curabiturnonvelitquistellustempussempernecalorem.Utbibendumorciaquammalesuada,nectinciduntduifacilisis.Nullamsollicitudinsitametligulainsodales.Vivamusfinibustempusmetusetbibendum.Aeneanvitaeultricesmauris.Donecimperdietdolornontellusimperdietsemper.Nuncquisfinibuslorem.Quisquevelodioblandit,variusfelisvitae,fermentumerat.Nuncfaucibusviverrametusvellacinia.Inatmagnalacus.Donecnongravidamassa.Craseuipsumodio.Inhachabitasseplateadictumst.Phaselluscondimentumefficitureros,eupretiumipsumeleifendquis.Vivamusegestas,maurisegetgravidafinibus,purusdiambibendumipsum,quispretiumnibhlacusacjusto.Duistortorquam,accumsanaccondimentumvitae,gravidautvolutpat.

becomes

23:53 <vulpineden> <foxie ๐ŸฆŠ ๐Ÿบ ๐Ÿฆฆ ๐Ÿ‰ ๐Ÿˆโฌ› ๐Ÿ† ๐Ÿฆจ ๐Ÿฆ> Pellentesqueluctustinciduntarcu,idportanequevolutpatut.Etiammagnafelis,pulvinaraerosin,molestieposuerearcu.Quisqueconsecteturnuncnonposueretincidunt.Curabiturnonvelitquistellustempussempernecalorem.Utbibendumorciaquammalesuada,nectinciduntduifacilisis.Nullamsollicitudinsitametligulainsodales.Vivamusfinibustempusmetusetbibendum.Aeneanvitaeultricesmauris.Donecimperdietdolornontellusimpe
23:53 <vulpineden> <foxie ๐ŸฆŠ ๐Ÿบ ๐Ÿฆฆ ๐Ÿ‰ ๐Ÿˆโฌ› ๐Ÿ† ๐Ÿฆจ ๐Ÿฆ> .Nuncfaucibusviverrametusvellacinia.Inatmagnalacus.Donecnongravidamassa.Craseuipsumodio.Inhachabitasseplateadictumst.Phaselluscondimentumefficitureros,eupretiumipsumeleifendquis.Vivamusegestas,maurisegetgravidafinibus,purusdiambibendumipsum,quispretiumnibhlacusacjusto.Duistortorquam,accumsanaccondimentumvitae,gravidautvolutpat.

remains rdietsemper

and

Menagerie.(๐Ÿงšโ™€*(๐ŸฆŠ๐Ÿ„๐Ÿ‰๐Ÿฐ๐Ÿฆ„๐ŸŒธ๐Ÿ’ป)) โ€” Today at 11:24 PM
00.0.00.00 - - [07/Nov/2022:20:34:04 -0700] "GET /packs/media/fonts/roboto/roboto-regular-webfont-e6505d5d85943244ec91d5e3002791f2.woff2 HTTP/2.0" 200 191468 "https://________.___/packs/css/flavours/glitch/common-1dba2fb7.chunk.css" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/000.00 (KHTML, like Gecko) Chrome/000.0.0.0 Safari/000.00 OPR/00.0.0000.000" - the roboto part was triggering it.  I need to wire up a better way to detect bots.
23:24 <%discordirc> <Menagerie.(๐Ÿงšโ™€*(๐ŸฆŠ๐Ÿ„๐Ÿ‰๐Ÿฐ๐Ÿฆ„๐ŸŒธ๐Ÿ’ป))> 00.0.00.00 - - [07/Nov/2022:20:34:04 -0700] "GET /packs/media/fonts/roboto/roboto-regular-webfont-e6505d5d85943244ec91d5e3002791f2.woff2 HTTP/2.0" 200 191468 "https://________.___/packs/css/flavours/glitch/common-1dba2fb7.chunk.css" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/000.00 (KHTML, like Gecko) Chrome/000.0.0.0 Safari/000.00 OPR/00.0.0000.000" - the roboto p

remains art was triggering it. I need to wire up a better way to detect bots.

note: domains, IP and versions scrubbed to match length to protect privacy.

A miscalculation occurs when a display name has emoji in it. ``` foxie ๐ŸฆŠ ๐Ÿบ ๐Ÿฆฆ ๐Ÿ‰ ๐Ÿˆโฌ› ๐Ÿ† ๐Ÿฆจ ๐Ÿฆ โ€” Today at 11:53 PM Pellentesqueluctustinciduntarcu,idportanequevolutpatut.Etiammagnafelis,pulvinaraerosin,molestieposuerearcu.Quisqueconsecteturnuncnonposueretincidunt.Curabiturnonvelitquistellustempussempernecalorem.Utbibendumorciaquammalesuada,nectinciduntduifacilisis.Nullamsollicitudinsitametligulainsodales.Vivamusfinibustempusmetusetbibendum.Aeneanvitaeultricesmauris.Donecimperdietdolornontellusimperdietsemper.Nuncquisfinibuslorem.Quisquevelodioblandit,variusfelisvitae,fermentumerat.Nuncfaucibusviverrametusvellacinia.Inatmagnalacus.Donecnongravidamassa.Craseuipsumodio.Inhachabitasseplateadictumst.Phaselluscondimentumefficitureros,eupretiumipsumeleifendquis.Vivamusegestas,maurisegetgravidafinibus,purusdiambibendumipsum,quispretiumnibhlacusacjusto.Duistortorquam,accumsanaccondimentumvitae,gravidautvolutpat. ``` becomes ``` 23:53 <vulpineden> <foxie ๐ŸฆŠ ๐Ÿบ ๐Ÿฆฆ ๐Ÿ‰ ๐Ÿˆโฌ› ๐Ÿ† ๐Ÿฆจ ๐Ÿฆ> Pellentesqueluctustinciduntarcu,idportanequevolutpatut.Etiammagnafelis,pulvinaraerosin,molestieposuerearcu.Quisqueconsecteturnuncnonposueretincidunt.Curabiturnonvelitquistellustempussempernecalorem.Utbibendumorciaquammalesuada,nectinciduntduifacilisis.Nullamsollicitudinsitametligulainsodales.Vivamusfinibustempusmetusetbibendum.Aeneanvitaeultricesmauris.Donecimperdietdolornontellusimpe 23:53 <vulpineden> <foxie ๐ŸฆŠ ๐Ÿบ ๐Ÿฆฆ ๐Ÿ‰ ๐Ÿˆโฌ› ๐Ÿ† ๐Ÿฆจ ๐Ÿฆ> .Nuncfaucibusviverrametusvellacinia.Inatmagnalacus.Donecnongravidamassa.Craseuipsumodio.Inhachabitasseplateadictumst.Phaselluscondimentumefficitureros,eupretiumipsumeleifendquis.Vivamusegestas,maurisegetgravidafinibus,purusdiambibendumipsum,quispretiumnibhlacusacjusto.Duistortorquam,accumsanaccondimentumvitae,gravidautvolutpat. ``` remains `rdietsemper` and ``` Menagerie.(๐Ÿงšโ™€*(๐ŸฆŠ๐Ÿ„๐Ÿ‰๐Ÿฐ๐Ÿฆ„๐ŸŒธ๐Ÿ’ป)) โ€” Today at 11:24 PM 00.0.00.00 - - [07/Nov/2022:20:34:04 -0700] "GET /packs/media/fonts/roboto/roboto-regular-webfont-e6505d5d85943244ec91d5e3002791f2.woff2 HTTP/2.0" 200 191468 "https://________.___/packs/css/flavours/glitch/common-1dba2fb7.chunk.css" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/000.00 (KHTML, like Gecko) Chrome/000.0.0.0 Safari/000.00 OPR/00.0.0000.000" - the roboto part was triggering it. I need to wire up a better way to detect bots. ``` ``` 23:24 <%discordirc> <Menagerie.(๐Ÿงšโ™€*(๐ŸฆŠ๐Ÿ„๐Ÿ‰๐Ÿฐ๐Ÿฆ„๐ŸŒธ๐Ÿ’ป))> 00.0.00.00 - - [07/Nov/2022:20:34:04 -0700] "GET /packs/media/fonts/roboto/roboto-regular-webfont-e6505d5d85943244ec91d5e3002791f2.woff2 HTTP/2.0" 200 191468 "https://________.___/packs/css/flavours/glitch/common-1dba2fb7.chunk.css" "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/000.00 (KHTML, like Gecko) Chrome/000.0.0.0 Safari/000.00 OPR/00.0.0000.000" - the roboto p ``` remains `art was triggering it. I need to wire up a better way to detect bots.` note: domains, IP and versions scrubbed to match length to protect privacy.
Poster
Owner

Likely caused by String.length/1 use instead of byte_size/1. Thankfully this will have a simple easy fix.

Likely caused by `String.length/1` use instead of `byte_size/1`. Thankfully this will have a simple easy fix.
foxiepaws self-assigned this 5 months ago
foxiepaws added spent time 5 months ago
5min
foxiepaws referenced this issue from a commit 5 months ago
foxiepaws closed this issue 5 months ago
Poster
Owner

it did not, in fact fix #4

it did not, in fact fix #4
foxiepaws reopened this issue 5 months ago
foxiepaws added the
bug
label 5 months ago
Poster
Owner

dff5ec0bde/lib/irc_bot.ex (L89)

probably involves this

https://git.foxiepa.ws/foxiepaws/discordirc/src/commit/dff5ec0bde6d560641b701fdcf26006e4c6c679e/lib/irc_bot.ex#L89 probably involves this
foxiepaws added spent time 5 months ago
2min
foxiepaws added spent time 5 months ago
4min
foxiepaws deleted spent time 5 months ago
- 4min
foxiepaws deleted spent time 5 months ago
- 2min
foxiepaws added spent time 5 months ago
5min
foxiepaws started working 5 months ago
foxiepaws stopped working 5 months ago
3h 15min 10s
foxiepaws added spent time 5 months ago
2min
foxiepaws started working 5 months ago
foxiepaws referenced this issue from a commit 5 months ago
foxiepaws closed this issue 5 months ago
foxiepaws stopped working 5 months ago
20min 11s
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Total Time Spent: 3h 47min 21s
Rachel Fae Fox
3h 47min 21s
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loadingโ€ฆ
There is no content yet.