the error is in line 97, the new function is called vichan_message_read. working now :-)

cheers

trw

Posted on def4

anon wrote:

> the error is in line 97, the new function is called vichan_message_read. working now :-)

Oops, forgot to change that, thanks!

--
Posted on: Rocksolid Light
www.novabbs.com

anon wrote:

>>Feel free to try the VICHAN command again on the test server nntp server.

> I could reach your server now, and tried it. It works well for articles with text only, for articles with attachments it returns nothing.

> See this, for example:

> vichan <ts.368.2r2bix@anon.com>

> cheers

> trw

It also currently only accepts an article number, not message-id. I didn't put a lot of effort into implementing the command until we decide whether we want it. If so, adding message-id is easy, I already do it for the article command.

--
Posted on: Rocksolid Light
www.novabbs.com

Retro Guy wrote:

> anon wrote:

>>>Feel free to try the VICHAN command again on the test server nntp server.

>> I could reach your server now, and tried it. It works well for articles with text only, for articles with attachments it returns nothing.

>> See this, for example:

>> vichan <ts.368.2r2bix@anon.com>

>> cheers

>> trw

> It also currently only accepts an article number, not message-id. I didn't put a lot of effort into implementing the command until we decide whether we want it. If so, adding message-id is easy, I already do it for the article command.

I have added ability to pull article by message-id for each command:
ARTICLE
BODY
HEAD
VICHAN

Also, I have applied your fix to vichan.php, thanks!

This is running on the test site right now.

Retro Guy

--
Posted on: Rocksolid Light
www.novabbs.com

Retro Guy wrote:

> Retro Guy wrote:

>> anon wrote:

>>>>Feel free to try the VICHAN command again on the test server nntp server.

>>> I could reach your server now, and tried it. It works well for articles with text only, for articles with attachments it returns nothing.

>>> See this, for example:

>>> vichan <ts.368.2r2bix@anon.com>

>>> cheers

>>> trw

>> It also currently only accepts an article number, not message-id. I didn't put a lot of effort into implementing the command until we decide whether we want it. If so, adding message-id is easy, I already do it for the article command.

> I have added ability to pull article by message-id for each command:
> ARTICLE
> BODY
> HEAD
> VICHAN

> Also, I have applied your fix to vichan.php, thanks!

> This is running on the test site right now.

I've just uploaded nntp.php and vichan.php to the download site. I moved this stuff into a subdir '/testing'.

vichan.php contains your fix.
nntp.php supports vichan, and search by message-id

Retro Guy

--
Posted on: Rocksolid Light
www.novabbs.com

a little update:

the good part: i could integrate your code, and together with the other lib i could build a solution that works locally without interfering with rslight (to be confirmed, of course).
that could be made into a solution that runs also without a local newsserver if vichan_message_read could get the article from a news server instead of reading it from the spool.
Also, I managed now to give the articles directly to vichan, without writing it to disk first and I can also capture the reply and do something with it, like syncing again later in case of errors (not implemented yet). All this makes the syncing faster and more reliable.
On the bad side, I have apparently trashed my vichan installation with all the hacking and trying i did, and I can't figure out what I broke exactly. So this will take some more time, until I know all the ins and outs of this software.

cheers

trw

Posted on def4