Issue 248 in glassdb: GRGemStonePlatform>>seasideDeliverEmailMessage is broken

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 248 in glassdb: GRGemStonePlatform>>seasideDeliverEmailMessage is broken

glassdb
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium GLASS-Server Version-GLASS0.231

New issue 248 by [hidden email]:  
GRGemStonePlatform>>seasideDeliverEmailMessage is broken
http://code.google.com/p/glassdb/issues/detail?id=248

Sending of emails is broken because  
GRGemStonePlatform>>seasideDeliverEmailMessage passes the 'to' email  
addresses to GemStone's SendMail as an array. Instead, it expects a string.

The fix is easy. Just replace the appropriate lines in  
GRGemStonePlatform>>seasideDeliverEmailMessage (see below)

        "deliver the mail"
        client
                mailhostName: self seasideSmtpServer;
                smtpPort: self seasideSmtpPort;
                from: aWAEmailMessage from address greaseString;
                to: (String streamContents:[:str |
                        aWAEmailMessage recipientsAddresses do:[:addr | str nextPutAll: addr  
greaseString] separatedBy: [str nextPut:$;]]);
                text: aWAEmailMessage plainMessage.

        client send.



Reply | Threaded
Open this post in threaded view
|

Re: Issue 248 in glassdb: GRGemStonePlatform>>seasideDeliverEmailMessage is broken

glassdb
Updates:
        Status: Accepted
        Owner: [hidden email]
        Labels: Milestone-1.0-beta.8.6

Comment #1 on issue 248 by [hidden email]:  
GRGemStonePlatform>>seasideDeliverEmailMessage is broken
http://code.google.com/p/glassdb/issues/detail?id=248

(No comment was entered for this change.)

Reply | Threaded
Open this post in threaded view
|

Re: Issue 248 in glassdb: GRGemStonePlatform>>seasideDeliverEmailMessage is broken

glassdb
Updates:
        Labels: -Milestone-1.0-beta.8.6 Milestone-3.0

Comment #2 on issue 248 by [hidden email]:  
GRGemStonePlatform>>seasideDeliverEmailMessage is broken
http://code.google.com/p/glassdb/issues/detail?id=248

will be making another Seaside30 pass soon ...

Reply | Threaded
Open this post in threaded view
|

Re: Issue 248 in glassdb: GRGemStonePlatform>>seasideDeliverEmailMessage is broken

glassdb
Updates:
        Labels: -Milestone-3.0 Milestone-1.0-beta.8.7

Comment #3 on issue 248 by [hidden email]:  
GRGemStonePlatform>>seasideDeliverEmailMessage is broken
http://code.google.com/p/glassdb/issues/detail?id=248

(No comment was entered for this change.)

Reply | Threaded
Open this post in threaded view
|

Re: Issue 248 in glassdb: GRGemStonePlatform>>seasideDeliverEmailMessage is broken

glassdb
Updates:
        Status: Fixed
        Labels: Fixed-1.0-beta.8.7

Comment #4 on issue 248 by [hidden email]:  
GRGemStonePlatform>>seasideDeliverEmailMessage is broken
http://code.google.com/p/glassdb/issues/detail?id=248

integrated changes into Seaside 3.0.5

Reply | Threaded
Open this post in threaded view
|

Re: Issue 248 in glassdb: GRGemStonePlatform>>seasideDeliverEmailMessage is broken

glassdb

Comment #5 on issue 248 by [hidden email]:  
GRGemStonePlatform>>seasideDeliverEmailMessage is broken
http://code.google.com/p/glassdb/issues/detail?id=248

See attached:

Name: SMTPMail-NickAger.9
Author: NickAger
Time: 13 June 2011, 17:53:53
UUID: a7a29d99-3c52-4af2-b911-c5b2d67a6553
Ancestors: SMTPMail-obi.7, SMTPMail-dkh.8

incorporated Esteban's changes to SendMail to allow authenticated smtp -  
smtps
see: http://forum.world.st/fixes-to-send-mails-with-gemstone-td3333512.html

and:
http://code.google.com/p/glassdb/issues/detail?id=248

Attachments:
        SMTPMail-NickAger.9.mcz  8.0 KB