Please use this thread to report bugs and give feedback. You may also start a thread on >>>/meta/, even though it's not an official board we will most likely still see it.Feel free to also report issues directly on our GitLab:https://gitlab.com/ison2/kotatsu
You are using get-image-dimensions on the fullthumb on the "((FLAC M4A MKV MP3 MP4 OCTET OGG WAV WMA WEBM) ; Audio/Video/Octet-streams" branchhttps://github.com/ECHibiki/Kotatsu-V-archived/blob/d953444684870b8c01d19b72f733db739ff42b07/prv/modules/imageboard.scm#L893instead of get-video-dimensions on the fullpath.https://github.com/ECHibiki/Kotatsu-V-archived/blob/d953444684870b8c01d19b72f733db739ff42b07/prv/modules/utils.scm#L107As a result video posts show the thumbnail dimensions instead of the video dimensions.https://4taba.net/thread/a/215#5p> 150x113
scheme@(guile-user)> (use-modules (srfi srfi-1))scheme@(guile-user)> (define options (string-split "nonoko sage" #\space))scheme@(guile-user)> options$1 = ("nonoko" "sage")scheme@(guile-user)> (define sage (lset-intersection equal? '("sage" "SAGE" "さ げ" "下げ") options))scheme@(guile-user)> sage$2 = ("sage")scheme@(guile-user)> (define nokosage (member "nokosage" options))scheme@(guile-user)> nokosage$3 = #fscheme@(guile-user)> (define sage-dbarg (or nokosage sage))scheme@(guile-user)> sage-dbarg$4 = ("sage")scheme@(guile-user)> (define name "Anonymous")scheme@(guile-user)> (define sage sage-dbarg)scheme@(guile-user)> (if (or (not sage) (null? sage) (equal? (car sage) "nokosage")) name (format #f "<a class=''sage'' href=''mailto:~a''>~a</a>" (car sage) name))$5 = "<a class=''sage'' href=''mailto:sage''>Anonymous</a>"scheme@(guile-user)> https://github.com/ECHibiki/Kotatsu-V/blob/cb5e6779821f0b3c919e2565e089a0901da2b3f8/prv/modules/imageboard.scm#L813https://github.com/ECHibiki/Kotatsu-V/blob/cb5e6779821f0b3c919e2565e089a0901da2b3f8/prv/modules/imageboard.scm#L960https://github.com/ECHibiki/Kotatsu-V/blob/cb5e6779821f0b3c919e2565e089a0901da2b3f8/prv/modules/database-calls.scm#L165
4taba has sage, but it's much easier to leave it disabled because I only spend 10 minutes a day here. Along with how there's no report functionality, means that it's safest to leave subposts as the sage-like posting method.
Another thing to consider is that 4taba with sage would mean that the site would look exceptionally inactive and not having it gives the site a bit more life, while taking away from the archive like nature that present 4taba has.
>>207So the code running on the site is not the one on github?
The thing is that the code is not made to further kotatsu as a project but meet the issue of easier moderation. I suppose if this is a concern I can make another repo for 4taba and one for kotatsu..https://github.com/ECHibiki/Kotatsu-V/blob/master/prv/modules/imageboard.scm
>>210> to further kotatsu as a projectYou can also further kotatsu with this fix: >>205
I'll look at what you're suggesting. I'm updating a bunch of bio stuff right now, but I've hit a good milestone on kissu to fix some issues here.I branched off the difference between what's ideally runable and my modifications for taba.https://github.com/ECHibiki
>>212Since you already have> (define-public noko-enabled #f)in settings.scm why not add a sage-enabled and control sage/nokosage in save-post with a conditional:(sage (if sage-enabled (lset-intersection equal? '("sage" "SAGE" "さげ" "下げ") options) '()))and similarly for nokosage. Also note that the negative return of member is #f rather than '().And you have> (define-public greeting "Shitaba Ni Yokouso<br/>")in settings.scmhttps://github.com/ECHibiki/Kotatsu-V/blob/cb5e6779821f0b3c919e2565e089a0901da2b3f8/prv/modules/settings.scm#L12but the word is Youkoso (ようこそ). Your u is misplaced.
Did you remove sage?
Here's a patch that adds iqdb links next to ImgOps links.
I'll review it today and add it
>>217 >>218Thanks for adding.
>>216Doesn't the imgops link already have iqdb in it?
>>220+ The separate iqdb link uses the thumbnail so you don't have to wait for the full image to transfer like for imgops when all you want is reverse lookup.+ The iqdb link inside imgops is in a place that is not immediately obvious to those who don't regularly use imgops, and it takes a second or two to scan for it, so the separate iqdb link provides convenience of access. To me the convenience of access was high enough to justify putting in the time to write and submit the patch.
> subpost creation before post exists> 20:51:28 test> 20:54:05 test #2> 20:55:51 subtestmuch keks
>>224It's the lack of a check for (string->number (car subpost)) throughout save-post in imageboard.scm.
>>225It's not a bug. It's a feature
A copy of thread 2 on the meta index sporting the title of thread 92.
I can't go directly to the catalog from a thread because there's no link, so I have to go through the board index every time. Here's a patch that adds a [Catalog] link to templates.scm:thread-tpl next to the [Return] link, using the same let approach as board-tpl. The custom order you had in thread-tpl between mod-bar-tpl and [Return] wasn't touched.
won''t add it today or tomorrow, but I'll put it in.Also remember that 4taba2 expires today and there will only be 4taba.net
Why is this image so blurry despite weighting 10 whole megabytes?
I'm performing backups to eventually do a server reinstall. I'll give some heads up so the site isn't down unexpectedly.
Site updates through email would give some greater visibility, I think
Tons of text turned into question marks.
backup is also before what I wanted. Did a backup before I wiped, but someone uploaded something that was tripping my antivirus causing me issues. Should have checked the filedates.The ???? aren't due to corruptionhttps://puu.sh/IBU4f/c7c07c5ec6.pngjust language pack issues. I want to finish off enabling the last two services and then I'll correct it. ?
think i had this issue two years back when I first set up the server. Unicode works perfectly fine on everything else but Guile or Artanis is getting confused.It may be the Guile database interface since the ???????? title at the top of the page renders fine. Suggesting that it's either the thing getting the information for templates rather than the templates themselves.
I give up on getting this to work with sqlite. I'm migrating the data to mysql
Nevermind, I got it. I seem to have remembered how I fixed it somewhat subconciously
I guess I'll add to the github that these steps will probably need to be followed on an ubuntu server installationhttps://askubuntu.com/questions/770309/cannot-permanently-change-locale-on-16-04-server
I have a messure to account for the 500 errors that got removed on reinstall
The current "Board link" lambda of imageboard.scm:comment-filterhttps://github.com/ECHibiki/Kotatsu-V/blob/f8b77f52ae2220cd9db3f3dc6191fec70191bcb1/prv/modules/imageboard.scm#L597has at least these problems: >>>/test/48/26+ the board name part will run over spaces and linefeeds due to "[^/]+"+ the extra character at the end will be pulled inside the link text, breaking up html entities like "+ the behavior at the end of the string is inconsistent due to the unconditional "(- (string-length (match:substring m)) 1)" in the let*Here's a patch that fixes these issues:+ the board name part no longer accepts spaces and linefeeds+ the extra character at the end goes after the closing tag where it belongs, preserving html entities+ the behavior at the end of the string is made consistent using match:substring 1Tests to verify the change are attached to the next post. I also think that code segments should be exempt from things like link markup, but fixing that would be a lot more work.
Here are the tests:1. the old way runs over spaces and breaks html entities2. the fix preserves html entities by placing the extra character at the end correctly after the closing tag3. the fix doesn't accept spaces4. the fix works consistently at the end of the string
>>243that's a strange oversight>has at least these problemsyeah
>>244 >>243ok, this is on my agenda nowI have to reinstall linux though
>>243Resolved,Site has also been updated to use Guile 3.0.5, DBI 2.1.8, Artanis 0.5.1 which have a few breaking changes. Most notably the database insertion doesn't seem to work the same way anymore. Guile related. Ports seemed to be getting appended to POST operations due to uri-path now not actually being the uri-path anymore(it was never actually a uri path anyways?). Artanis related.URLs for image files changed to remove /pub/. Should have been like this in the first place.Otherwise unimportant.
So, since the software has been updated there may be a few 500 errors around that I haven't seen.
>>249>Resolved,Thanks.Since your post has iqdb with no image I think the thumb test in templates.scm:iqdb-tpl needs the same new test as imgops-tpl. Since the test changed several times, it might as well be factored out so it can be controlled from one place.As a bit of trivia, you made all your static png, html and similar files executable; just making sure you're aware.
Have to switch it back on the repository. it's 664 on the live. Yes, posts without files are no longer being given a boolean from the DB writter, and I'm not motivated read into why this is so I'm writting null strings instead of #f
Also archive.4taba seems to be having some issues. Could have been in January or yesterday. I'll see about fixing it tonight.
was nginx configuration
You're getting thumbnail dimensions in >>>/test/39/38 due to >>205.
I find myself wanting to switch to another listed board's catalog, but I keep having to go through the board's index page first, or having to edit the board name in the address bar, which gets tedious. It would be more convenient for me if the board list provided catalog links as well.Here's a patch to add unobtrusive catalog links to div#top-nav, in addition to the existing index links.
Can something be done about the spam please?
>>257I can try two things. As long as it doesn't pile up though.>>256Doesn't look too obtrusive
cloudflare's maximum settings reduce the spam from 10 hits per day to 3
Now lets see what happens with cloudflare set to "essentially off"
>>260Looks like cloudflare has a massive impact on simple spam bots. Alternative solution will require false post fields