-
src/sbbs3/js_msgbase.c
From
deuce@VERT to
CVS commit on Monday, September 30, 2013 19:03:27
src/sbbs3 js_msgbase.c 1.184 1.185
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv16816
Modified Files:
js_msgbase.c
Log Message:
Fix last commit... was still using the offset, just wasn't an array.
this one really does use the message number now.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ
telnet://vert.synchro.net
-
From
deuce@VERT to
CVS commit on Saturday, October 05, 2013 07:29:22
src/sbbs3 js_msgbase.c 1.185 1.186
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv4521
Modified Files:
js_msgbase.c
Log Message:
Remove unused variable.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ
telnet://vert.synchro.net
-
From
deuce@VERT to
CVS commit on Saturday, November 08, 2014 12:18:59
src/sbbs3 js_msgbase.c 1.187 1.188
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv26575
Modified Files:
js_msgbase.c
Log Message:
Fix bug in parsing header objects which prevented being able to set thread_back.
The get of thread_orig would succeed, leaving val as undefined so the get of thread_back would never occur. Testing *after* this for null or undefined would result in thread_back never being set in the parsed header.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, October 27, 2015 18:24:09
src/sbbs3 js_msgbase.c 1.189 1.190
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20116
Modified Files:
js_msgbase.c
Log Message:
Remove obsolete crash report/stack trace. This bug was fix over a year ago now.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, November 26, 2015 02:56:06
src/sbbs3 js_msgbase.c 1.192 1.193
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv22495
Modified Files:
js_msgbase.c
Log Message:
Make the SMB plain-text only feature available to users of the MsgBase.get_msg_body() method: added an additional boolean argument (following the 'include_tails' argument), which defaults to false (disabled). Pass true for this argument to enable this feature. Yes, JS-authors could do this parsing easily via JS, but I figure so long as this feature is in the SMBLIB, might as well expose it to JS for consistency (e.g. in 3rd party message reader mods).
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ
telnet://vert.synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, August 18, 2017 21:47:06
src/sbbs3 js_msgbase.c 1.206 1.207
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
js_msgbase.c
Log Message:
Create MsgBase.close_poll() method - used to close an existing poll
the arguments are: MessageNumber, UserNameOrAlias
The MessageNumber is the message number for the close to be polled.
The UserNameOrAlias should be the currentl authenticated user and must
match the pollster (sender) of the poll for the closure to succeed.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, November 01, 2017 15:41:06
src/sbbs3 js_msgbase.c 1.207 1.208
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv1479
Modified Files:
js_msgbase.c
Log Message:
Hopefully fix issue with converting auxattr with bit 31 set to/from JS value: Poll with results "closed" (only visible to pollster until the poll is closed) have bit 31 set in the auxattr message header field. Reportedly (by Nightfox) this causes Error: can't convert 2147483648 to an integer when attempting
to use this message in JS.
I could'vd sworn I've solved this problem before, but maybe that was with
JS-C v1.5 - anyway, first convert to double using JS_ValueToNumber and then typecast to uint32.
We probably will need to do something similar with the time values in year
2038 (holding my breath).
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Argon@VERT/STARFLT to
rswindell on Thursday, November 02, 2017 22:35:18
Re: src/sbbs3/js_msgbase.c
By: rswindell to CVS commit on Wed Nov 01 2017 03:41 pm
We probably will need to do something similar with the time values in year 2038 (holding my breath).
And I'll be 75 then... lol.
I'll be able to say, "I've been BBSing for 50 years." in 2038.
Darryl
---
þ Synchronet þ StarFlight BBS - bbs.starflightbbs.com
-
From
rswindell@VERT to
CVS commit on Tuesday, November 21, 2017 18:44:15
src/sbbs3 js_msgbase.c 1.209 1.210
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
js_msgbase.c
Log Message:
Fixed bug in JS msgbase.save_msg(), when multiple recipients were
passed (as an array), the message's data blocks would be allocated with a refcount of 1, as though only one message header referenced the data. This
is really only an issue with bulkmail/multiple-recipient email and when the first message was deleted, the remaining (headers) would point to free
data blocks which could and eventually would be overwritten by another message's data (body text). chksmb would only catch the problem when the refcount reached 0 but there was still one or more message headers pointing
at the data.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, November 27, 2017 14:34:53
src/sbbs3 js_msgbase.c 1.211 1.212
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv8361
Modified Files:
js_msgbase.c
Log Message:
Fix a bug in the get_msg_header() method introduced in rev 1.195 (12-Nov-2016): The 'expand_fields' argument wasn't be parsed correctly and defaults to 'true', so any script that modifies a message header could be writing-back expandeed header fields which may sometimes fail depending on how much the total header size expanded.
Enhancement: the put_msg_header() method will now fail with an exception if a script tries to write-back a message header that was read with expanded fields. This should help to avoid potential message base corruption through a script bug.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, November 27, 2017 17:00:26
src/sbbs3 js_msgbase.c 1.212 1.213
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv23177
Modified Files:
js_msgbase.c
Log Message:
Fix a potential null pointer dereference in the last commit.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, November 30, 2017 20:52:08
src/sbbs3 js_msgbase.c 1.213 1.214
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
js_msgbase.c
Log Message:
Fix likely cause of crash in MsgBase.get_msg_header(String id):
incrementing the argv index in a call to JSSTRING_TO_MSTRING causes the
index to be incremented twice!
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Friday, January 19, 2018 20:14:56
src/sbbs3 js_msgbase.c 1.214 1.215
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv7266
Modified Files:
js_msgbase.c
Log Message:
Added missing mention of msg hdr.can_read property
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, February 22, 2018 02:29:28
src/sbbs3 js_msgbase.c 1.217 1.218
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv10027
Modified Files:
js_msgbase.c
Log Message:
Fix double-free bug introduced in js_remove_msg() in previous commit
(Coverity clean-up) - apparently this function was already erroneously freeing the object private pointer (p) and I copy/pasted that error into more places.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, May 15, 2018 15:41:59
src/sbbs3 js_msgbase.c 1.219 1.220
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv17644
Modified Files:
js_msgbase.c
Log Message:
Fix get_msg_header() problem reported by Bill McGarrity:
"expand fields" could be misinterpretted (e.g. as 'false') if less than 3
args were passed to the function. Apparently you can NOT assume that argv[argc] is undefined and would fail a JSVAL_IS_BOOLEAN test. In the reported problem, MsgBase.get_msg_header() was being called with 2 arguments (from newslink.js) and the if(JSVAL_IS_BOOLEAN(argv[n])) test, when n was 2, would eval to true and then argv[n] evalulated as false, which would cause a message with no message ID to not have one dynamically created, which would then cause the message to fail to post to an NNTP server due to malformed Message-ID (a missing message "id" property would end up being included in the newsgropu article header as "Message-ID: undefined").
get_msg_index() had a similar potential issue, also fixed.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, July 15, 2018 22:42:08
src/sbbs3 js_msgbase.c 1.220 1.221
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv8385
Modified Files:
js_msgbase.c
Log Message:
Rename this private_t.status to private_t.smb_result, to avoid confusion
with smb_t.status.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, October 03, 2018 01:13:19
src/sbbs3 js_msgbase.c 1.222 1.223
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv25520
Modified Files:
js_msgbase.c
Log Message:
Added "tags" to the message header property fields populated when getting
a message header.
Added "tags" and "summary" (previously missing) to the property fields
parsed when adding/writing a message header. This parse code is getting particulary copy/pasta and should be converted to a table-driven loop
reusing the same logic.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, January 21, 2019 02:10:32
src/sbbs3 js_msgbase.c 1.223 1.224
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv20155
Modified Files:
js_msgbase.c
Log Message:
Attempt to clarify all this MsgBase.get...(by_offset, number_or_offset_or_id_header)
confusion: You should not pass both a message number *and* a message header to the
retrieval methods that support referring to a message by either (number/offset, id
or header).
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, January 21, 2019 19:18:04
src/sbbs3 js_msgbase.c 1.224 1.225
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv16237
Modified Files:
js_msgbase.c
Log Message:
Beautification/typo of MsgBase methods by_offset blurb.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, February 16, 2019 03:38:30
src/sbbs3 js_msgbase.c 1.225 1.226
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv24228
Modified Files:
js_msgbase.c
Log Message:
MsgBase.get_all_msg_headers() performance improvement:
read the entire index file in a single read operation
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, February 17, 2019 19:14:48
src/sbbs3 js_msgbase.c 1.226 1.227
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv26214
Modified Files:
js_msgbase.c
Log Message:
Added support for new(ish) msg header fields: 'editor' and 'columns'.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, February 19, 2019 04:13:51
src/sbbs3 js_msgbase.c 1.227 1.228
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
js_msgbase.c
Log Message:
Fix long-standing bug in js_ParseMsgHeaderObject which is only (currently)
used by js_post_msg() (the JS bbs.post_msg() method when used with the reply_header object argument) - the private data attached to a message
header object is of type privatemsg_t, not private_t. This caused the dereferences of and assignments to p->smb_result in parse_header_object()
to corrupt the privatemsg_t->msg memory causing heap corruption (caught
on Windows debug builds in js_get_msg_header_finalize()) and stack faults (caught on Linux-gcc in parse_recipient_object()). This one was hard to
find. <whew!>
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, May 02, 2019 17:27:56
src/sbbs3 js_msgbase.c 1.244 1.245
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv1076
Modified Files:
js_msgbase.c
Log Message:
Document the message header fields:
- votes
- priority
- delivery_attempts
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Tuesday, July 16, 2019 01:04:31
src/sbbs3 js_msgbase.c 1.248 1.249
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv14339
Modified Files:
js_msgbase.c
Log Message:
Use Spidermonkey's JS_ValueToECMAUint32() rather than our own JS_ValueToUint32() for converting to integers where we may need the full 32 bits.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Wednesday, July 24, 2019 02:29:03
src/sbbs3 js_msgbase.c 1.249 1.250
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv22884
Modified Files:
js_msgbase.c
Log Message:
New read-only msg header property populated by get_msg_header() method:
- Boolean: is_utf8
In the case of multi-part MIME messages, this may not be populated
accurately until the message body is read with the get_msg_body() method.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Monday, August 05, 2019 22:04:30
src/sbbs3 js_msgbase.c 1.250 1.251
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv14631
Modified Files:
js_msgbase.c
Log Message:
Handle setting msg.hdr.delivery_attempts in parse_header_object()
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, August 22, 2019 21:50:06
src/sbbs3 js_msgbase.c 1.251 1.252
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv23668
Modified Files:
js_msgbase.c
Log Message:
Add ftn_charset property for message headers. This header field corresponds with the FTN (FTS-5003) "CHRS" control line/paragraph. The values recoginized by Synchronet are:
"ASCII 1"
"CP437 2"
"UTF-8 4"
These values indicate that header fields and body text of a message are
encoded with the specifiec charset. The default (assumed charset, if not specified), is CP437.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, February 29, 2020 23:52:14
src/sbbs3 js_msgbase.c 1.252 1.253
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv10423
Modified Files:
js_msgbase.c
Log Message:
Bug-fix: when enumerating a message header (returned from MsgBase.get_msg_header or get_all_msg_headers), the message header object's private data (used internally in these C source functions) would be freed and NULL'd, apparenty as a form of optimization. This would cause some methods which can accept a msg header object as an argument (e.g. MsgBase.put_msg_header, bbs.show_msg/show_msg_header) to fail or behave
in strange ways.
Instead of freeing/NULLing the private data (and depending on that as an indication that the header has been enuemrated), just set a member variable indicating that the header has been enumerated (once) already.
This is the bug that has been tripping me up with my message lister JS mod
(see YouTube video). I can finally get that committed to CVS for testing now. :-)
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
rswindell on Sunday, March 01, 2020 11:10:32
Re: src/sbbs3/js_msgbase.c
By: rswindell to CVS commit on Sat Feb 29 2020 11:52 pm
This is the bug that has been tripping me up with my message lister JS mod (see YouTube video). I can finally get that committed to CVS for testing now. :-)
w00t! :)
With something like that more officially built into Synchronet, I was considering whether it would be worth it to continue maintaining my own message reader..
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Sunday, March 01, 2020 11:47:11
Re: src/sbbs3/js_msgbase.c
By: Nightfox to rswindell on Sun Mar 01 2020 11:10 am
Re: src/sbbs3/js_msgbase.c
By: rswindell to CVS commit on Sat Feb 29 2020 11:52 pm
This is the bug that has been tripping me up with my message lister JS mod (see YouTube video). I can finally get that committed to CVS for testing now. :-)
w00t! :)
With something like that more officially built into Synchronet, I was considering whether it would be worth it to continue maintaining my own message reader..
Well, it's not exactly "built-in". And it's certainly still a proof of concept, so unlikely to be a good candidate for full-time use just yet. Also, I don't think it's going to replace message-scans like your reader can/does. Or at least, not yet. Right now, it's a functions pretty well to read mail and list/search/sort messages in a sub, but not useful for new-msg-scans.
digital man
Synchronet/BBS Terminology Definition #15:
CVS = Concurrent Versioning System
Norco, CA WX: 57.8øF, 57.0% humidity, 4 mph WSW wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Digital Man@VERT to
Nightfox on Sunday, March 01, 2020 12:43:55
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Sun Mar 01 2020 11:47 am
Re: src/sbbs3/js_msgbase.c
By: Nightfox to rswindell on Sun Mar 01 2020 11:10 am
Re: src/sbbs3/js_msgbase.c
By: rswindell to CVS commit on Sat Feb 29 2020 11:52 pm
This is the bug that has been tripping me up with my message lister JS mod (see YouTube video). I can finally get that committed to CVS for testing now. :-)
w00t! :)
With something like that more officially built into Synchronet, I was considering whether it would be worth it to continue maintaining my own message reader..
Well, it's not exactly "built-in". And it's certainly still a proof of concept, so unlikely to be a good candidate for full-time use just yet. Also, I don't think it's going to replace message-scans like your reader can/does. Or at least, not yet. Right now, it's a functions pretty well to read mail and list/search/sort messages in a sub, but not useful for new-msg-scans.
Anything I could do to encourage you to continue your message reader development? It's definitely beneficial to have other developers trying things and attacking problems from a different angle. So I'm certainly not trying to compete with your mod or disaude you from continuing to use/improve upon it. I hope you do.
digital man
Synchronet "Real Fact" #63:
"Baja" (name of Synchronet PCMS compiler/languege) is pronounced "ba-ha". Norco, CA WX: 53.9øF, 72.0% humidity, 8 mph NNE wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Sunday, March 01, 2020 14:21:58
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Sun Mar 01 2020 11:47 am
With something like that more officially built into Synchronet, I was
considering whether it would be worth it to continue maintaining my
own message reader..
Well, it's not exactly "built-in".
True.. I guess "built-in" wasn't the best choice of words; I just meant something more official included with Synchronet.
And it's certainly still a proof of
concept, so unlikely to be a good candidate for full-time use just yet. Also, I don't think it's going to replace message-scans like your reader can/does. Or at least, not yet. Right now, it's a functions pretty well to
Would it be an easy thing to support using Synchronet's stock message scan, but to be able to just swap out the mesage reader interface with a .js script? The difference I'd see useful there would be to be able to support scrolling up & down in the message (for ANSI users).
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Sunday, March 01, 2020 14:34:54
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Sun Mar 01 2020 12:43 pm
Anything I could do to encourage you to continue your message reader development? It's definitely beneficial to have other developers trying things and attacking problems from a different angle. So I'm certainly not trying to compete with your mod or disaude you from continuing to use/improve upon it. I hope you do.
I know you aren't trying to compete. The main reason I created my message reader was to provide a scrolling interface when reading messages. If that will be part of an officially-included JS script, I actually have no problem with that. And as the author of Synchronet, you are very familiar with the C/C++ source code, so you could probably more easily optimize that code to help the JS scripts perform better, if there are optimizations that can be done.
There are some other features I've added to my reader, and I don't remember now if the stock Synchronet reader does all these:
- The ability to forward a message to a user/email address (I've used this for sending a Dove-Net message to myself to remind myself of something someone said about Synchronet, etc.)
- The ability to save a message to the BBS machine (for sysops)
- Users can download attachments in their personal email
- The ability to switch between the reader and the list of messages in the current sub-board and choose another message to read
- The ability to change to another sub-board from the reader or message list and read/list messages in the other chosen sub-board
Also I like to implement a lightbar interface where possible, for ANSI users (and for non-ANSI terminals, I like to fall back to a more traditional interface).
I can't think of a whole lot that could help with continued development. One thing though is that when I list the messages, I use get_all_msg_headers() so I can filter out the voting replies, deleted messages, etc., and that can be a little slow if there are many messages in a sub-board. I'm not sure if there's much that can be done to help that though..
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
rswindell@VERT to
CVS commit on Sunday, March 01, 2020 15:51:25
src/sbbs3 js_msgbase.c 1.253 1.254
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv11059
Modified Files:
js_msgbase.c
Log Message:
JS_GetClass() can return NULL. Don't dereference the return value without checking for non-NULL value.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Digital Man@VERT to
Nightfox on Sunday, March 01, 2020 16:34:17
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Sun Mar 01 2020 02:21 pm
And it's certainly still a proof of
concept, so unlikely to be a good candidate for full-time use just yet. Also, I don't think it's going to replace message-scans like your reader can/does. Or at least, not yet. Right now, it's a functions pretty well to
Would it be an easy thing to support using Synchronet's stock message scan, but to be able to just swap out the mesage reader interface with a .js script? The difference I'd see useful there would be to be able to support scrolling up & down in the message (for ANSI users).
I've thought about that as well, but no, not easy. The main reason is that to scroll back through a message, the script needs to be parsing keyboard input (e.g. for arrow keys, page up/dn, etc.) rather than accepting a command key (e.g. 'Q' for quit, Enter for next message, etc.). The command key input in the "stock message scan" is in the C++ source. So you can't combine a JS scroll-back with a C++ command-key/input parsing. The control is either in the JS module or it's in the C++ code, never both simultaneously.
If the JS module did all the command-key parsing, then would have to handle those keys and thus be able to move to another message (e.g. thread-forward/back), find text in messages, list headers, etc. And now you're just back to a full-fledge message reader module.
I suppose that a script could parse all the message scrolling keys and then just return to the BBS the value of any other key that is hit so that those other command keys could then be acted upon by the existing C++ code. It'd be bouncing back and forth between and JS and C++ a lot and it's a bit akward to be returning a key value from a script (but not impossible).
Anyway, it's likely possible (really, anything is possible), but I don't see it being "easy".
digital man
Synchronet/BBS Terminology Definition #32:
FTS = FidoNet Technical Standard
Norco, CA WX: 51.0øF, 75.0% humidity, 1 mph N wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Digital Man@VERT to
Nightfox on Sunday, March 01, 2020 16:39:50
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Sun Mar 01 2020 02:34 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Sun Mar 01 2020 12:43 pm
Anything I could do to encourage you to continue your message reader development? It's definitely beneficial to have other developers trying things and attacking problems from a different angle. So I'm certainly not trying to compete with your mod or disaude you from continuing to use/improve upon it. I hope you do.
I know you aren't trying to compete. The main reason I created my message reader was to provide a scrolling interface when reading messages. If that will be part of an officially-included JS script, I actually have no problem with that. And as the author of Synchronet, you are very familiar with the C/C++ source code, so you could probably more easily optimize that code to help the JS scripts perform better, if there are optimizations that can be done.
Yeah and when I find that there isn't JS support for something nice to have or needed, I usually add it right away and make use of it.
There are some other features I've added to my reader, and I don't remember now if the stock Synchronet reader does all these:
- The ability to forward a message to a user/email address (I've used this for sending a Dove-Net message to myself to remind myself of something someone said about Synchronet, etc.)
Not exactly. You can (M)ail reply to yourself though.
- The ability to save a message to the BBS machine (for sysops)
Yup.
- Users can download attachments in their personal email
Yup (wouldn't it be weird if they couldn't?)
- The ability to switch between the reader and the list of messages in the current sub-board and choose another message to read
Well, that's in my msglist mod. The stock message scan/read interface doesn't really have a "mode" to switch between.
- The ability to change to another sub-board from the reader or message list and read/list messages in the other chosen sub-board
Nope.
Also I like to implement a lightbar interface where possible, for ANSI users (and for non-ANSI terminals, I like to fall back to a more traditional interface).
Yup, me too. Just more work. :-)
I can't think of a whole lot that could help with continued development.
One thing though is that when I list the messages, I use get_all_msg_headers() so I can filter out the voting replies, deleted messages, etc., and that can be a little slow if there are many messages in a sub-board. I'm not sure if there's much that can be done to help that though..
get_all_msg_headers() is the fastest way to load all the message headers. There is an argument for the automatic filtering of votes. And that's what I use in my msglist module.
digital man
Synchronet "Real Fact" #39:
Synchronet first supported Windows NT v6.x (a.k.a. Vista/Win7) w/v3.14a (2006). Norco, CA WX: 51.0øF, 75.0% humidity, 1 mph N wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
DaiTengu@VERT/ENSEMBLE to
Nightfox on Monday, March 02, 2020 08:02:48
Re: src/sbbs3/js_msgbase.c
By: Nightfox to rswindell on Sun Mar 01 2020 11:10 am
w00t! :)
With something like that more officially built into Synchronet, I was considering whether it would be worth it to continue maintaining my own message reader..
Please continue. Until your message reader came along, I used to have to use GoldEd, or download everthing as a QWK packet.
The only things I wish it would do is print some sort of status while it's scanning message bases, and have the ability to reply to a message in a different message area (I came across the latter one today, actually)
Otherwise it's the bees knees, and I thank you for it!
DaiTengu
... I haven't lost my mind; it's backed up on tape somewhere!
---
þ Synchronet þ War Ensemble BBS - The sport is war, total war - warensemble.com
-
From
Nightfox@VERT/DIGDIST to
DaiTengu on Monday, March 02, 2020 12:16:15
Re: src/sbbs3/js_msgbase.c
By: DaiTengu to Nightfox on Mon Mar 02 2020 08:02 am
Please continue. Until your message reader came along, I used to have to use GoldEd, or download everthing as a QWK packet.
The only things I wish it would do is print some sort of status while it's scanning message bases,
I'm not sure what you mean? While doing a newscan etc., it does print the sub-board names while it's scanning.
and have the ability to reply to a message in a
different message area (I came across the latter one today, actually)
I'm not sure what you mean by this? When you reply to a message, you will be replying to the current message in whatever sub-board it's in. Do you mean some sort of cross-post feature? A cross-post wouldn't be a feature of a message reader. With SlyEdit, you can cross-post, and with the cross-posting feature, you could also change the sub-board that your reply will be posted in, in case you want it to be posted in a more appropriate sub-board.
Otherwise it's the bees knees, and I thank you for it!
I'm glad you enjoy using it. It seems like Digital Man might be incorporating similar functionality though.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Monday, March 02, 2020 14:26:54
Re: src/sbbs3/js_msgbase.c
By: Nightfox to DaiTengu on Mon Mar 02 2020 12:16 pm
I'm not sure what you mean by this? When you reply to a message, you will be replying to the current message in whatever sub-board it's in. Do you mean some sort of cross-post feature? A cross-post wouldn't be a feature of a message reader. With SlyEdit, you can cross-post, and with the cross-posting feature, you could also change the sub-board that your reply will be posted in, in case you want it to be posted in a more appropriate sub-board.
How's that work? From sbbs's perspective, an external message editor (e.g. SlyEdit) is just editing a text file (the message text) and possibly a drop file (e.g. result.ed). I'm not clear how you can change the area where a message is being saved from an external message editor.
digital man
This Is Spinal Tap quote #33:
Nigel Tufnel: Well, so what? What's wrong with bein' sexy?
Norco, CA WX: 65.0øF, 27.0% humidity, 6 mph SW wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Monday, March 02, 2020 15:24:54
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 02:26 pm
Do you mean some sort of cross-post feature? A cross-post wouldn't be
a feature of a message reader. With SlyEdit, you can cross-post, and
with the cross-posting feature, you could also change the sub-board
that your reply will be posted in, in case you want it to be posted in
a more appropriate sub-board.
How's that work? From sbbs's perspective, an external message editor (e.g. SlyEdit) is just editing a text file (the message text) and possibly a drop file (e.g. result.ed). I'm not clear how you can change the area where a message is being saved from an external message editor.
SlyEdit lets you cross-post by selecting additional areas to post a message in (useful if you want to post a message in different message networks, for instance). SlyEdit uses MsgBase.save_msg() to post the message to sub-boards other than the one currently being posted in. If you want to just change the area where your message is being posted, you can de-select the current sub-board and select another sub-board, and SlyEdit will use MsgBase.save_msg() to post in the other sub-board and will then clear the current message and return an "aborted" code so that Synchronet won't post it in the original sub-board.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Monday, March 02, 2020 16:43:13
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Mon Mar 02 2020 03:24 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 02:26 pm
Do you mean some sort of cross-post feature? A cross-post wouldn't be
a feature of a message reader. With SlyEdit, you can cross-post, and
with the cross-posting feature, you could also change the sub-board
that your reply will be posted in, in case you want it to be posted in
a more appropriate sub-board.
How's that work? From sbbs's perspective, an external message editor (e.g. SlyEdit) is just editing a text file (the message text) and possibly a drop file (e.g. result.ed). I'm not clear how you can change the area where a message is being saved from an external message editor.
SlyEdit lets you cross-post by selecting additional areas to post a message in (useful if you want to post a message in different message networks, for instance). SlyEdit uses MsgBase.save_msg() to post the message to sub-boards other than the one currently being posted in. If you want to just change the area where your message is being posted, you can de-select the current sub-board and select another sub-board, and SlyEdit will use MsgBase.save_msg() to post in the other sub-board and will then clear the current message and return an "aborted" code so that Synchronet won't post it in the original sub-board.
Ewww.... Okay, I guess that "works". So you wouldn't get Synchronet-added signatures and other things that happen when a message is posted may not happen. So I would expect some weird behavior and corner-cases when using that feature of your editor.
digital man
Synchronet/BBS Terminology Definition #77:
TTY = Teletype (dumb terminal)
Norco, CA WX: 65.8øF, 27.0% humidity, 0 mph SW wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Monday, March 02, 2020 17:15:38
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 04:43 pm
SlyEdit lets you cross-post by selecting additional areas to post a
message in (useful if you want to post a message in different message
networks, for instance). SlyEdit uses MsgBase.save_msg() to post the
message to sub-boards other than the one currently being posted in.
If you want to just change the area where your message is being
posted, you can de-select the current sub-board and select another
sub-board, and SlyEdit will use MsgBase.save_msg() to post in the
other sub-board and will then clear the current message and return an
"aborted" code so that Synchronet won't post it in the original
sub-board.
Ewww.... Okay, I guess that "works". So you wouldn't get Synchronet-added signatures and other things that happen when a message is posted may not happen. So I would expect some weird behavior and corner-cases when using that feature of your editor.
When cross-posting to other sub-boards that way, SlyEdit will read the user's signature file and append it to the message - so yes, it will get the user's Synchronet signature. The cross-posting feature was something requested from another sysop, and that was the only way I could think of to do it. I'm not sure if there's a better way..
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Monday, March 02, 2020 20:12:14
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Mon Mar 02 2020 05:15 pm
SlyEdit lets you cross-post by selecting additional areas to post a
message in (useful if you want to post a message in different message
networks, for instance). SlyEdit uses MsgBase.save_msg() to post the
message to sub-boards other than the one currently being posted in.
If you want to just change the area where your message is being
posted, you can de-select the current sub-board and select another
sub-board, and SlyEdit will use MsgBase.save_msg() to post in the
other sub-board and will then clear the current message and return an
"aborted" code so that Synchronet won't post it in the original
sub-board.
Ewww.... Okay, I guess that "works". So you wouldn't get
Synchronet-added signatures and other things that happen when a
message is posted may not happen. So I would expect some weird
behavior and corner-cases when using that feature of your editor.
If Synchronet had a JS function or some mechanism to cross-post a user's message to other sub-boards and do everything Synchronet normally does when posting a user's message, that could be useful to SlyEdit.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Monday, March 02, 2020 21:55:41
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Mon Mar 02 2020 05:15 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 04:43 pm
SlyEdit lets you cross-post by selecting additional areas to post a
message in (useful if you want to post a message in different message
networks, for instance). SlyEdit uses MsgBase.save_msg() to post the
message to sub-boards other than the one currently being posted in.
If you want to just change the area where your message is being
posted, you can de-select the current sub-board and select another
sub-board, and SlyEdit will use MsgBase.save_msg() to post in the
other sub-board and will then clear the current message and return an
"aborted" code so that Synchronet won't post it in the original
sub-board.
Ewww.... Okay, I guess that "works". So you wouldn't get Synchronet-added signatures and other things that happen when a message is posted may not happen. So I would expect some weird behavior and corner-cases when using that feature of your editor.
When cross-posting to other sub-boards that way, SlyEdit will read the user's signature file and append it to the message - so yes, it will get the user's Synchronet signature. The cross-posting feature was something requested from another sysop, and that was the only way I could think of to do it. I'm not sure if there's a better way..
Right now, there is no "better way" (because Synchronet doesn't have that feature), but messages posted in that fashion are going to be noticeabley different from messages posted "normally", main with regards to the header fields added by sbbs when saving a message and the message text post-processing that happens.
It's probably better to discuss design decisions like that before you go in a "creative" direction. :-) I probably could add some specific support for that feature (maybe in a drop file?) had I known that someone wanted or needed it.
Are you doing proper security checking in your cross-posting support?
digital man
Synchronet/BBS Terminology Definition #71:
SyncTERM = The Synchronet Terminal Emulator/BBS-client program
Norco, CA WX: 61.9øF, 20.0% humidity, 5 mph W wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Digital Man@VERT to
Nightfox on Monday, March 02, 2020 22:00:14
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Mon Mar 02 2020 08:12 pm
If Synchronet had a JS function or some mechanism to cross-post a user's message to other sub-boards and do everything Synchronet normally does when posting a user's message, that could be useful to SlyEdit.
Yeah, I'd never thought about that before. Simply changing which sub-board via a drop file could be done "pretty" easy (not posting to multiple subs). But there are some decisions that are made based on the sub-board's configuration, before the external editor is even invoked. For example, could someone cross post a message into an Anonymous sub-board? Or a private-only sub-board? If yes, would the message be posted anonymously or privately? Can an email be cross-posted into a sub-board? These seem like potentially problematic scenarios.
digital man
Synchronet "Real Fact" #27:
Rob Swindell (digital man) was born approximately 4 hours before the Unix epoch.
Norco, CA WX: 61.9øF, 20.0% humidity, 5 mph W wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Tuesday, March 03, 2020 14:28:35
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 09:55 pm
When cross-posting to other sub-boards that way, SlyEdit will read the
user's signature file and append it to the message - so yes, it will
get the user's Synchronet signature. The cross-posting feature was
something requested from another sysop, and that was the only way I
could think of to do it. I'm not sure if there's a better way..
Right now, there is no "better way" (because Synchronet doesn't have that feature), but messages posted in that fashion are going to be noticeabley different from messages posted "normally", main with regards to the header fields added by sbbs when saving a message and the message text post-processing that happens.
It's probably better to discuss design decisions like that before you go in a "creative" direction. :-) I probably could add some specific support for that feature (maybe in a drop file?) had I known that someone wanted or needed it.
Often I try to work with what is provided in the JS API. I saw that MsgBase.post_msg() was available, and at the time I didn't think there would need to be special support added for user message posting. In the future I'll plan on checking with you on things like that.
Are you doing proper security checking in your cross-posting support?
As far as a user being able to post? I thought it was, but it looks like I may have gone in the assumption that the sub-board would not be shown available sub-boards to the user if they weren't able to post there.. I'll have to update SlyEdit to ensure the user has permissions to post before doing so.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Tuesday, March 03, 2020 14:30:14
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 10:00 pm
If Synchronet had a JS function or some mechanism to cross-post a
user's message to other sub-boards and do everything Synchronet
normally does when posting a user's message, that could be useful to
SlyEdit.
Yeah, I'd never thought about that before. Simply changing which sub-board via a drop file could be done "pretty" easy (not posting to multiple subs). But there are some decisions that are made based on the sub-board's
SlyEdit allows posting to multiple sub-boards (i.e., if the message could fit into sub-boards on multiple message networks)..
configuration, before the external editor is even invoked. For example, could someone cross post a message into an Anonymous sub-board? Or a private-only sub-board? If yes, would the message be posted anonymously or privately? Can an email be cross-posted into a sub-board? These seem like potentially problematic scenarios.
SlyEdit checks to see if a sub-board is anonymous and whether to use the user's real name or handle before posting.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Tuesday, March 03, 2020 16:14:50
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Tue Mar 03 2020 02:28 pm
Are you doing proper security checking in your cross-posting
support?
As far as a user being able to post? I thought it was, but it looks like I may have gone in the assumption that the sub-board would not be shown available sub-boards to the user if they weren't able to post there.. I'll have to update SlyEdit to ensure the user has permissions to post before doing so.
Upon closer look, for cross-posting, SlyEdit already does not allow the user to select a sub-board that they don't have posting permission for.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Tuesday, March 03, 2020 16:52:02
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Tue Mar 03 2020 02:28 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 09:55 pm
When cross-posting to other sub-boards that way, SlyEdit will read the
user's signature file and append it to the message - so yes, it will
get the user's Synchronet signature. The cross-posting feature was
something requested from another sysop, and that was the only way I
could think of to do it. I'm not sure if there's a better way..
Right now, there is no "better way" (because Synchronet doesn't have that feature), but messages posted in that fashion are going to be noticeabley different from messages posted "normally", main with regards to the header fields added by sbbs when saving a message and the message text post-processing that happens.
It's probably better to discuss design decisions like that before you go in a "creative" direction. :-) I probably could add some specific support for that feature (maybe in a drop file?) had I known that someone wanted or needed it.
Often I try to work with what is provided in the JS API. I saw that MsgBase.post_msg() was available, and at the time I didn't think there would need to be special support added for user message posting. In the future I'll plan on checking with you on things like that.
Are you doing proper security checking in your cross-posting support?
As far as a user being able to post? I thought it was, but it looks like I may have gone in the assumption that the sub-board would not be shown available sub-boards to the user if they weren't able to post there.. I'll have to update SlyEdit to ensure the user has permissions to post before doing so.
The user is not "shown" the JS object model normally, so I'm not sure what mechanism you're to provide the user the option of selecting another sub-board to post to. It's certainly possible to have sub-board that a user can read but not post to. That said, the JS msg_area.grp_list and sub_list arrays are filtered based on "access" (not posting) privileges. The msg_area.sub and msg_area.grp arrays are not filtered based on access (contain all subs/groups on the system, regardless of which is user is currently logged int).
digital man
Synchronet/BBS Terminology Definition #66:
SpiderMonkey = Mozilla's C/C++ JavaScript Engine (libmozjs)
Norco, CA WX: 74.3øF, 18.0% humidity, 2 mph NNW wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Digital Man@VERT to
Nightfox on Tuesday, March 03, 2020 16:57:53
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Tue Mar 03 2020 02:30 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Mon Mar 02 2020 10:00 pm
If Synchronet had a JS function or some mechanism to cross-post a
user's message to other sub-boards and do everything Synchronet
normally does when posting a user's message, that could be useful to
SlyEdit.
Yeah, I'd never thought about that before. Simply changing which sub-board via a drop file could be done "pretty" easy (not posting to multiple subs). But there are some decisions that are made based on the sub-board's
SlyEdit allows posting to multiple sub-boards (i.e., if the message could fit into sub-boards on multiple message networks)..
There are sub-board settings that are used to post-process the message text created by an external editor, so the same message text may not be suitable (without modification) for multiple sub-boards, depending on the sysop's configuration.
configuration, before the external editor is even invoked. For example, could someone cross post a message into an Anonymous sub-board? Or a private-only sub-board? If yes, would the message be posted anonymously or privately? Can an email be cross-posted into a sub-board? These seem like potentially problematic scenarios.
SlyEdit checks to see if a sub-board is anonymous and whether to use the user's real name or handle before posting.
Cool, that's something. :-)
digital man
Synchronet/BBS Terminology Definition #35:
HTTP = Hypertext Transfer Protocol
Norco, CA WX: 74.3øF, 18.0% humidity, 2 mph NNW wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Tuesday, March 03, 2020 17:32:56
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Tue Mar 03 2020 04:52 pm
The user is not "shown" the JS object model normally, so I'm not sure what mechanism you're to provide the user the option of selecting another sub-board to post to. It's certainly possible to have sub-board that a user can read but not post to. That said, the JS msg_area.grp_list and sub_list arrays are filtered based on "access" (not posting) privileges. The msg_area.sub and msg_area.grp arrays are not filtered based on access (contain all subs/groups on the system, regardless of which is user is currently logged int).
SlyEdit iterates through msg_area.grp_list and then msg_area.grp_list.sub_list to allow the user to select sub-boards to post into. After checking earlier, I remembered SlyEdit already doesn't allow the user to select a sub-board they can't post in.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Tuesday, March 03, 2020 21:30:44
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Tue Mar 03 2020 05:32 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Tue Mar 03 2020 04:52 pm
The user is not "shown" the JS object model normally, so I'm not sure what mechanism you're to provide the user the option of selecting another sub-board to post to. It's certainly possible to have sub-board that a user can read but not post to. That said, the JS msg_area.grp_list and sub_list arrays are filtered based on "access" (not posting) privileges. The msg_area.sub and msg_area.grp arrays are not filtered based on access (contain all subs/groups on the system, regardless of which is user is currently logged int).
SlyEdit iterates through msg_area.grp_list and then msg_area.grp_list.sub_list to allow the user to select sub-boards to post into. After checking earlier, I remembered SlyEdit already doesn't allow the user to select a sub-board they can't post in.
Okay, that's good. What method are using to determine if they can or can't post in a sub?
digital man
This Is Spinal Tap quote #41:
Ian Faith: It say's "Memphis show cancelled due to lack of advertising funds." Norco, CA WX: 62.6øF, 33.0% humidity, 1 mph SSE wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Tuesday, March 03, 2020 21:51:48
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Tue Mar 03 2020 09:30 pm
post into. After checking earlier, I remembered SlyEdit already
doesn't allow the user to select a sub-board they can't post in.
Okay, that's good. What method are using to determine if they can or can't post in a sub?
This line of code does the check (msgSubCode is the sub-board code):
if (user.compare_ars(msg_area.sub[msgSubCode].post_ars))
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Wednesday, March 04, 2020 01:38:08
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Tue Mar 03 2020 09:51 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Tue Mar 03 2020 09:30 pm
post into. After checking earlier, I remembered SlyEdit already
doesn't allow the user to select a sub-board they can't post in.
Okay, that's good. What method are using to determine if they can or can't post in a sub?
This line of code does the check (msgSubCode is the sub-board code):
if (user.compare_ars(msg_area.sub[msgSubCode].post_ars))
Yeah, that's insufficient. For example, that doesn't check weither the user has the 'N' restriction and the sub-board is a networked sub. Instead, you should be checking "can_post" property value of the sub.
digital man
Synchronet "Real Fact" #73:
SyncTERM (created by Deuce) contains portions of Synchronet and SEXYZ code. Norco, CA WX: 55.0øF, 46.0% humidity, 0 mph S wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rampage@VERT/SESTAR to
Nightfox on Wednesday, March 04, 2020 07:23:12
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Mon Mar 02 2020 17:15:38
The cross-posting feature was something requested from another
sysop, and that was the only way I could think of to do it.
I'm not sure if there's a better way..
cross-posting and replying in another area, including the netmail/email area, are features that are certainly missing in the base sbbs offerings... there's been numerous times that i've wanted to reply to someone via netmail or in another area to avoid being offtopic or to provide an answer in private...
)\/(ark
---
þ Synchronet þ The SouthEast Star Mail HUB - SESTAR
-
From
DaiTengu@VERT/ENSEMBLE to
Nightfox on Wednesday, March 04, 2020 11:30:53
Re: src/sbbs3/js_msgbase.c
By: Nightfox to DaiTengu on Mon Mar 02 2020 12:16 pm
The only things I wish it would do is print some sort of status
while it's scanning message bases,
I'm not sure what you mean? While doing a newscan etc., it does print the sub-board names while it's scanning.
Hmm, it's possible I'm using an older version. When I do a newscan It just hangs at a black screen with a cursor until it finds an area that has new messages. I should probably give my message reading system a once over to verify everything's up to date.
and have the ability to reply to a message in a
different message area (I came across the latter one today,
actually)
I'm not sure what you mean by this? When you reply to a message, you will be replying to the current message in whatever sub-board it's in. Do you mean some sort of cross-post feature? A cross-post wouldn't be a feature of a message reader. With SlyEdit, you can cross-post, and with the cross-posting feature, you could also change the sub-board that your reply will be posted in, in case you want it to be posted in a more appropriate sub-board.
Yeah, I somehow spaced on the cross-post option. Apologies. Working 12-16 hour days is killing me, and shredding what little sanity I have left.
I'm glad you enjoy using it. It seems like Digital Man might be incorporating similar functionality though.
But, that requires change! I don't like change!
DaiTengu
... Modesty is a vastly overrated virtue.
---
þ Synchronet þ War Ensemble BBS - The sport is war, total war - warensemble.com
-
From
Digital Man@VERT to
Rampage on Wednesday, March 04, 2020 10:25:16
Re: src/sbbs3/js_msgbase.c
By: Rampage to Nightfox on Wed Mar 04 2020 07:23 am
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Mon Mar 02 2020 17:15:38
The cross-posting feature was something requested from another
sysop, and that was the only way I could think of to do it.
I'm not sure if there's a better way..
cross-posting and replying in another area, including the netmail/email area, are features that are certainly missing in the base sbbs offerings... there's been numerous times that i've wanted to reply to someone via netmail or in another area to avoid being offtopic or to provide an answer in private...
You can already reply via netmail with the 'M' command.
digital man
Synchronet/BBS Terminology Definition #64:
SMB = Synchronet Message Base (e.g. smblib)
Norco, CA WX: 64.5øF, 36.0% humidity, 0 mph ESE wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rampage@VERT/SESTAR to
Digital Man on Wednesday, March 04, 2020 13:43:15
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Rampage on Wed Mar 04 2020 10:25:16
You can already reply via netmail with the 'M' command.
i'll be damned! you can! i'm used to that stuff showing in the prompt when reading a message... something like
[N]ext, [P]revious, [R]eply, [M]ail, [O]ther
and similar... RA's methods are still ingrained and will take a while to me moved out and filled with sbbs methods LUL
)\/(ark
---
þ Synchronet þ The SouthEast Star Mail HUB - SESTAR
-
From
Digital Man@VERT to
Rampage on Wednesday, March 04, 2020 11:10:44
Re: src/sbbs3/js_msgbase.c
By: Rampage to Digital Man on Wed Mar 04 2020 01:43 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Rampage on Wed Mar 04 2020 10:25:16
You can already reply via netmail with the 'M' command.
i'll be damned! you can! i'm used to that stuff showing in the prompt when reading a message... something like
[N]ext, [P]revious, [R]eply, [M]ail, [O]ther
There are too many options to list them all on a prompt.
digital man
Synchronet "Real Fact" #91:
Captured chat with Wayne Bell:
http://wiki.synchro.net/history:waynebell_chat Norco, CA WX: 68.9øF, 34.0% humidity, 2 mph SSE wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
DaiTengu@VERT/ENSEMBLE to
Rampage on Wednesday, March 04, 2020 11:34:27
Re: src/sbbs3/js_msgbase.c
By: Rampage to Nightfox on Wed Mar 04 2020 07:23 am
cross-posting and replying in another area, including the netmail/email area, are features that are certainly missing in the base sbbs offerings... there's been numerous times that i've wanted to reply to someone via netmail or in another area to avoid being offtopic or to provide an answer in private...
Yeah, it hasn't been an issue until.... recently. There are a lot of messages from a particular user where I've just wanted to reply in a different, more appropriate message area.
DaiTengu
... Life shouldn't be printed on dollar bills.
---
þ Synchronet þ War Ensemble BBS - The sport is war, total war - warensemble.com
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Wednesday, March 04, 2020 12:35:15
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Wed Mar 04 2020 01:38 am
This line of code does the check (msgSubCode is the sub-board code):
if (user.compare_ars(msg_area.sub[msgSubCode].post_ars))
Yeah, that's insufficient. For example, that doesn't check weither the user has the 'N' restriction and the sub-board is a networked sub. Instead, you should be checking "can_post" property value of the sub.
I'll change it to use can_post. But I'm curious why it matters if it's a networked sub? Are there certain conditions where a user wouldn't be able to post on a networked sub?
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Nightfox@VERT/DIGDIST to
DaiTengu on Wednesday, March 04, 2020 12:38:24
Re: src/sbbs3/js_msgbase.c
By: DaiTengu to Nightfox on Wed Mar 04 2020 11:30 am
I'm not sure what you mean? While doing a newscan etc., it does
print the sub-board names while it's scanning.
Hmm, it's possible I'm using an older version. When I do a newscan It just hangs at a black screen with a cursor until it finds an area that has new messages. I should probably give my message reading system a once over to verify everything's up to date.
My reader always printed the sub-board names when doing a scan. However, one time after updating my Synchronet binaries, I noticed it doesn't always print all the sub-board names (for me on my BBS). I'm not sure what changed that may have affected that.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Wednesday, March 04, 2020 13:00:12
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Wed Mar 04 2020 12:35 pm
Yeah, that's insufficient. For example, that doesn't check weither
the user has the 'N' restriction and the sub-board is a networked
sub. Instead, you should be checking "can_post" property value of
the sub.
I'll change it to use can_post. But I'm curious why it matters if it's a networked sub? Are there certain conditions where a user wouldn't be able to post on a networked sub?
I see what you're saying now. You can disregard my question..
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
mark lewis@VERT to
Nightfox on Wednesday, March 04, 2020 16:11:14
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Wed Mar 04 2020 12:35:15
Yeah, that's insufficient. For example, that doesn't check
weither the user has the 'N' restriction and the sub-board is
a networked sub. Instead, you should be checking "can_post"
property value of the sub.
I'll change it to use can_post. But I'm curious why it
matters if it's a networked sub? Are there certain
conditions where a user wouldn't be able to post on a networked
sub?
consider a sysop message area where only sysops are allowed to post but anyone may read...
then you have areas in (eg) fidonet that are restricted to only certain zone, regions, or nets... i'm sure there are other examples where one may read an area but not post in it...
)\/(ark
--- SBBSecho 3.10-Linux
* Origin: SouthEast Star Mail HUB - SESTAR (1:3634/12)
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Nightfox@VERT/DIGDIST to
Digital Man on Wednesday, March 04, 2020 13:41:08
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Wed Mar 04 2020 01:38 am
This line of code does the check (msgSubCode is the sub-board code):
if (user.compare_ars(msg_area.sub[msgSubCode].post_ars))
Yeah, that's insufficient. For example, that doesn't check weither the user has the 'N' restriction and the sub-board is a networked sub. Instead, you should be checking "can_post" property value of the sub.
Some of the JS documentation could probably be a little clearer on things like that.
http://www.synchro.net/docs/jsobjs.html
For can_post, the documentation says "user has sufficient access to post messages". That doesn't really sound different from checking post_ars with the user's access requirements. It might be good to have more explanation of things like that in the JS docs so people developing JS mods can have a better idea of when to use each thing.
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Nightfox@VERT/DIGDIST to
mark lewis on Wednesday, March 04, 2020 15:08:55
Re: src/sbbs3/js_msgbase.c
By: mark lewis to Nightfox on Wed Mar 04 2020 04:11 pm
I'll change it to use can_post. But I'm curious why it
matters if it's a networked sub? Are there certain
conditions where a user wouldn't be able to post on a networked
sub?
consider a sysop message area where only sysops are allowed to post but anyone may read...
Yep. That could be done by setting the ARS string to SYSOP. I found that checking the user's ARS would be sufficient in that case..
then you have areas in (eg) fidonet that are restricted to only certain zone, regions, or nets... i'm sure there are other examples where one may read an area but not post in it...
I see..
Nightfox
---
þ Synchronet þ Digital Distortion: digitaldistortionbbs.com
-
From
Digital Man@VERT to
Nightfox on Wednesday, March 04, 2020 16:25:51
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Wed Mar 04 2020 12:35 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Wed Mar 04 2020 01:38 am
This line of code does the check (msgSubCode is the sub-board code):
if (user.compare_ars(msg_area.sub[msgSubCode].post_ars))
Yeah, that's insufficient. For example, that doesn't check weither the user has the 'N' restriction and the sub-board is a networked sub. Instead, you should be checking "can_post" property value of the sub.
I'll change it to use can_post. But I'm curious why it matters if it's a networked sub? Are there certain conditions where a user wouldn't be able to post on a networked sub?
Yes, if the user has the 'N' restriction.
Also, if the user has the 'P' restriction, they can't post in any sub-board.
And, if the user has already posted their maximum number of posts-per-day allowed (based on their security level), they can't post in any sub-board.
digital man
Synchronet/BBS Terminology Definition #62:
SEXYZ = Synchronet External X/Y/ZMODEM file transfer protocol driver
Norco, CA WX: 67.9øF, 60.0% humidity, 6 mph SE wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Digital Man@VERT to
Nightfox on Wednesday, March 04, 2020 16:27:41
Re: src/sbbs3/js_msgbase.c
By: Nightfox to Digital Man on Wed Mar 04 2020 01:41 pm
Re: src/sbbs3/js_msgbase.c
By: Digital Man to Nightfox on Wed Mar 04 2020 01:38 am
This line of code does the check (msgSubCode is the sub-board code):
if (user.compare_ars(msg_area.sub[msgSubCode].post_ars))
Yeah, that's insufficient. For example, that doesn't check weither the user has the 'N' restriction and the sub-board is a networked sub. Instead, you should be checking "can_post" property value of the sub.
Some of the JS documentation could probably be a little clearer on things like that.
http://www.synchro.net/docs/jsobjs.html
For can_post, the documentation says "user has sufficient access to post messages". That doesn't really sound different from checking post_ars with the user's access requirements. It might be good to have more explanation of things like that in the JS docs so people developing JS mods can have a better idea of when to use each thing.
True. Developers can and should also look at the existing *.js mods for example best practices to follow.
digital man
Synchronet "Real Fact" #26:
The Synchronet Web Server was written predominantly by Stephen Hurd (Deuce). Norco, CA WX: 67.9øF, 60.0% humidity, 6 mph SE wind, 0.00 inches rain/24hrs
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rampage@VERT/SESTAR to
Nightfox on Wednesday, March 04, 2020 18:52:04
Re: src/sbbs3/js_msgbase.c
By: Nightfox to mark lewis on Wed Mar 04 2020 15:08:55
I'll change it to use can_post. But I'm curious why it
matters if it's a networked sub? Are there certain
conditions where a user wouldn't be able to post on a
networked sub?
consider a sysop message area where only sysops are allowed
to post but anyone may read...
Yep. That could be done by setting the ARS string to SYSOP.
wrong sysop... "sysops" in my above meant "visiting sysops"... sorry...
)\/(ark
---
þ Synchronet þ The SouthEast Star Mail HUB - SESTAR
-
From
rswindell@VERT to
CVS commit on Wednesday, March 18, 2020 00:05:38
src/sbbs3 js_msgbase.c 1.254 1.255
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv23819
Modified Files:
js_msgbase.c
Log Message:
Fix bug in MsgBase.get_index() and get_all_msg_headers():
The number of messages loaded was limited by the number of messages that were in the status header the last time the status header was read (i.e. when the msgbase was opened or when any of its status header properties were referenced).
Also fixed a couple of weird/useless uses of the WHERE macro in error messages.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, March 19, 2020 12:41:33
src/sbbs3 js_msgbase.c 1.255 1.256
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv10236
Modified Files:
js_msgbase.c
Log Message:
Create MsgBase.dump_msg_header() method which will perform a low-level header "dump" to an array of strings using the (new) smb_msghdr_str_list() SMBLIB function. The message header passed must've been previously returned from
the get_msg_header() or get_all_msg_headers() method.
A similar dump could be constructed from the header properties, as available in JS, but that would be inconsistent with other msg header dumps and likely lack header fields or details available from the lowest level dump method provided in SMBLIB.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Saturday, April 04, 2020 15:07:05
src/sbbs3 js_msgbase.c 1.256 1.257
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv1155
Modified Files:
js_msgbase.c
Log Message:
Bug: a get_mg_header() followed by a put_msg_header() would add a header field (SMTPREVERSEPATH a.k.a. "reverse_path") if it didn't already have one.
That's because smb_getmsghdr() will point the msg.reverse_path convenience pointer to the from_net_addr if there was no explicit reverse-path (e.g. RFC822's
"return-path" header field). This could manifest itself in *any* change to a message header via JS failing with an "illegal header length increase" error if the added header field just happen to put the total header length over the allocation threshold of the pre-existing msg header.
Fix: only model a msg header "reverse_path" property if the header field actually existed (not based on the SMBLIB convenience pointer).
When the expand_fields option is used, the old behavior remains but expanded headers cannot be written back to the base, so no harm there.
Reported by Coz in #synchronet from failed runs of scrubmsgs.js. Thanks!
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Sunday, April 05, 2020 22:18:01
src/sbbs3 js_msgbase.c 1.257 1.258
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv9320
Modified Files:
js_msgbase.c
Log Message:
MsgBase.open() would not, could not, actually create a message base.
It would create 3 0-byte files (*.shd, *.sdt, *.sid), but more is actually needed for a message base to be "created" (i.e. a call to smb_create()).
So, MsgBase.open() now uses smb_open_sub() rather than smb_open() to initialize theSMB status fields with the proper default values (based on the sysop configuration) and calls smb_crate() if the header file is empty.
Yes, normally, SCFG creates message bases, but it shouldn't have to
(e.g. a fresh install on *nix, doesn't actually start with any files in data/subs) and now that we have JavaScript-based message lister/readers, we really needed this support.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, April 23, 2020 22:08:03
src/sbbs3 js_msgbase.c 1.259 1.260
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv8971
Modified Files:
js_msgbase.c
Log Message:
The RECIPIENTLIST header field would get removed (converted to an RFC822TO header field) when saving a message using the MsgBase class.
A similar problem existed with REPLYTOLIST/RFC822REPLYTO, but was not actually observed.
Since the following header fields were not populated in the msg header "field_list", if they existed in a message header that was modified using
the MsgBase class, they would be lost:
- RFC822TO
- RFC822CC
- RFC822ORG
- RFC822REPLYTO
- RFC822SUBJECT
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, May 07, 2020 12:29:10
src/sbbs3 js_msgbase.c 1.260 1.261
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/tmp/cvs-serv3023
Modified Files:
js_msgbase.c
Log Message:
Allow an added or modified message header to have to/from/reply-to "net type" values of NET_NONE. This is a weird scenario, but it is technically possible (e.g. for nntpservice.js) to save a message with a to/from/treply-to "net address" header, but an associated net-type of NET_NONE. By saving the net-type NET_NONE, when saving a modified header, if the associated net address header field value cannot be parsed into a valid network address, there won't be any error reported, e.g.
Error -110 adding SENDERNETADDR field to message header
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
rswindell@VERT to
CVS commit on Thursday, May 07, 2020 14:58:38
src/sbbs3 js_msgbase.c 1.261 1.262
Update of /cvsroot/sbbs/src/sbbs3
In directory cvs:/home/rswindell/sbbs/src/sbbs3
Modified Files:
js_msgbase.c
Log Message:
Populate the *_net_type fields, even when set to NET_NONE (0), when the corresponding *_net_addr field is present (not NULL).
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to sbbs/master on Sunday, November 22, 2020 00:14:40
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Saturday, May 22, 2021 21:44:37
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Saturday, July 10, 2021 22:58:26
https://gitlab.synchro.net/main/sbbs/-/commit/e53c5926508c739c27c4d67c
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
Ignore the PRIVATE message attribute for the "mail" base
When setting the value of a message's 'can_read' property, ignore the PRIVATE message attribute (which is sometimes set in FTN netmail messages) since it's assumed all messages in the mail base are private, no special destination (to) name matching is needed here.
This only popped up recently via msglist.js because of the recent addition of checking each messages's 'can_read' property.
As reported by <Diehard> via IRC PM.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, April 17, 2022 21:20:59
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, April 21, 2022 22:09:52
https://gitlab.synchro.net/main/sbbs/-/commit/1e62752ab8340af66f18f0b0
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
Restore ability for MsgBase.open() to open an arbitrary SMB msgbase
Before commit 5da26eca, you could pass Msgbase() the path to an SMB
on the disk and open() it, no configuration needed. As of 2 years
ago, I broke that, and passing a path to an SMB would open the "mail" base instead - most unexpected. This is a feature of smb_open_sub() which we switched to using (from smb_open()), so go back to using smb_open() when
an unrecognized code is pass to the constructor.
This has the negative consequence that the "mail" base can't be created
via JS. Probably should fix that.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Thursday, April 21, 2022 22:09:52
https://gitlab.synchro.net/main/sbbs/-/commit/dc4b1bd9350a5581fa410c7f
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
Use smb_open_sub() for the "mail" base too
This restores the ability for JS MsgBase() to be used to create the
initial mail message base properly, if needed.
This means that the 'subnum' should now be equal to scfg.total_subs
when referencing an arbitrary SMB via path (not in the configuration).
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Sunday, May 08, 2022 11:31:19
https://gitlab.synchro.net/main/sbbs/-/commit/323ff1e8198df77405409ab2
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
Remember the last 'first_msg' property value after msgbase is closed
The other msgbase property values are stored in the smb.status or other private_t members which retain their value when the message base is closed, unlike the 'first_msg' property which required a read of the message base index to get the value (and that doesn't work when the message base is closed, so would return 0).
for deon (ALTERANT)
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell@VERT to
Git commit to main/sbbs/master on Saturday, November 19, 2022 17:02:14
https://gitlab.synchro.net/main/sbbs/-/commit/6c5cb01c0979e74332c314e4
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
Fix return return value when methods auto-open a message base (e.g. save_msg())
js_open() sets the rval for the context (e.g. to JSVAL_TRUE), so we need to set it back to JSVAL_FALSE for failure conditions. Otherwise, the methods always return true even upon (save-msg) failure.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Wednesday, March 29, 2023 12:52:05
https://gitlab.synchro.net/main/sbbs/-/commit/fe14b001e563ea3b10c82cbf
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
MsgBase.save_msg() throw an exception when an empty recipient list is provided
... rather than just return false.
This will make debugging this type of issue much easier in the future. sendmail.js was allowing empty recipient_list array arguments and the MsgBase .status was 0 and .error just an emtpy string - unhelpful.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Sunday, May 07, 2023 23:22:44
https://gitlab.synchro.net/main/sbbs/-/commit/d5e8981d0631d766b37a061f
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
MsgBase.save_msg() docs says only one argument is required
So use js_argc() to report an error if there's fewer than 1 argument, but
don't just fail silently when 2 arguments are provided.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows)@VERT to
Git commit to main/sbbs/master on Monday, May 08, 2023 18:07:50
https://gitlab.synchro.net/main/sbbs/-/commit/de41e1b7a314ceaacfb6c321
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
Use js_argc() to generate the helpful exception when insufficient args used
... rather than just failing silently with a special return value
(e.g. false).
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Wednesday, December 13, 2023 19:11:22
-
From
Rob Swindell (on Windows 11)@VERT to
Git commit to main/sbbs/master on Saturday, March 02, 2024 16:15:48
https://gitlab.synchro.net/main/sbbs/-/commit/82b9cbe85972cfdbd1ff5255
Modified Files:
src/sbbs3/js_msgbase.c
Log Message:
More use of JS_ValueToECMAUint32 in place of JS_ValueToInt32
To support the full 32-bit range for things like dates/times, and message numbers (!).
This should fix issue #732
Though dates beyond Feb 7 2106 are definitely going to still be an issue. I'll definitley look into solving that by the the 2100.
---
þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net