Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

Questions

  • (tick) Should entities be allowed to have more than one associated message thread? 
    • A:  Yes.  A "thread" is just a linked list of messages, linked by their inReplyTo reference, which is optional.  The ID of the thread is just the ID of the first message in the list, which has an empty 'inReplyTo' field.  So if more than one message is 'sent to' an entity (as part of a discussion forum) with an empty inReplyTo field, then there is more than one thread for the entity.
  • (error) Or message threads to have more than one associated entity?
  • (error) Should we allow messages to be sent to multiple non-principals at once?
  • Should messages with low numbers of recipients be processed in an immediately consistent manner?
    • (tick) Sending messages to non-principals (i.e. commenting on an entity) does not need asynchronous processing
    • (tick) Sending messages to a single recipient will be done transactionally
  • What ACCESS_TYPE should be associated with the ability to comment on an entity?
    • (tick) SEND_MESSAGE
  • What ACCESS_TYPE should be associated with the ability to message a user?
    • (tick) No restriction
    • (tick) Add a blacklist
    • (tick) Flag as inappropriate
  • Should the worker use SQS or RDS to manage the flow of messages to send?
    • (question) Proposed RDS implementation:
      • Add a migratable table with a single column of message IDs
      • Add a worker that periodically polls the table
      • If the table is not empty, process N rows from the top
      • Delete rows once finished processing
    • Using SQS opens up the possibility of losing messages (especially during weekend stack switching).  The state stored in SQS would not be migrated between stacks.  And unlike change messages, there is no trivial method to detect if a message has been changed.  And reusing the change messages is not possible, since retransmission of messages is incorrect behavior. 
  • How should we bounce messages?  Silently?  Via an auto-generated message?
      (tick) Add services to check if message can be sent.  This way, the UI can check if a message can be sent before sending it.  
    • (question) On an error, send an error message to the sender's inbox.
  • How should we handle messages sent to groups rather than individuals?  Should the message be broken into individuals messages (after checking for SEND_MESSAGE permission on the group)?  (question) Currently, it simply stashes the message, leaving no way of fetching the message (other than as a sender).  

Objects

NameDBOMigration

DTO

  •  MessageContent

(Immutable after creation)

  • Primary key: ID
  • Foreign key: CreatedBy (UserGroup)
  • Foreign key: MessageBody (S3 File Handle ID)
  • Etag
  • CreatedOn

Backup via ID

Note:  Etag is required because MessageStatus is mutable.

Interface

  • ID
  • CreatedBy (principal ID)
  • S3 File Handle ID
  • CreatedOn
  •  MessageToUser

(Immutable after creation)

  • Foreign key: ID (MessageContent)
  • Subject (nullable)
  • Self foreign key: InReplyTo (ID) (nullable)
  • Foreign key: InReplyToRootId
    • Recursive foreign key to primary key

Secondary to MessageContent

Implements MessageContent

  • Subject (string, optional)
  • List of recipient IDs
  • inReplyTo (optional)
  •  MessageRecipient

(Immutable after creation)

  • Foreign key: ID (MessageContent)
  • Foreign key: Recipient ID (UserGroup)

Secondary to MessageContent

 
  •  MessageStatus
  • Foreign key: ID (MessageContent)
  • Foreign key: Recipient ID (UserGroup)
  • States: UNREAD, READ, ARCHIVED

Secondary to MessageContent

  • ID
  • Recipient
  • State
  •  Comment

(Immutable after creation)

  • Foreign key: ID (MessageContent)
  • Target type (Enum ObjectType)
  • Target ID
  • TBD:  references to other Comments in the conversation*

Secondary to MessageContent

Implements MessageContent

  • Target type
  • Target ID
  •  Message votesCommentVotes
  • Primary key: Message ID
  • Up votes
  • Down votes
  • Is-comment-inappropriate votes
TBD
  • ID
  • Up
  • Down
  •  MessageSettingsMessagingSettings
  • Foreign key: User ID (UserGroup)
  • Etag
  • Blob of settings
TBD
  • ID
  • Notification levels
    • Update to an owned entity
    • Update to some favorite entity
    • Team
    • Message from user
    • Mail to everyone
    • Etc...
  • Message receipt email (if more than one)
  • Auto-mark mail as read if forwarded to email
  • Blacklist of blocked users
  •  MessageBundle
  Bundles the MessageToUser and Message Status

...

Other

  •  Message ID generator
  •  Thread ID generator
  •  Email mechanism(tick) Use Amazon SES Message delivery will be via worker, i.e. asynchronous (in most cases) with respect to message creation
    •  Messages to a low number (1) of recipients should be immediately consistent
    •  Comments should always be immediately consistent
    • (question) Proposed RDS implementation
      • Add a migratable table with a single column of message IDs
      • Add a worker that periodically polls the table
      • If the table is not empty, process N rows from the top
      • Delete rows once finished processing
  •  Email mechanism (use Amazon SES)
  •  Message delivery will be via worker, i.e. asynchronous (in most cases) with respect to message creation.

...

Services

  • Sort by recipient
MethodURIunderlying query (if any)BodyParametersReturnDescriptionPermission
  •  GET
/message/inboxselect * from messagecontent c, messagetouser m, messagestatus s where c.ID=m.messageContentID and s.messageContentID=c.ID and s.recipientID=<pid> 

Sorting + pagination

Paginated results<MessageBundle>Gets all messages the authenticated user has receivedAuthenticated User
  •  GET
/message/outbox

select * from messagecontent c, messagetouser m where c.ID=m.messageContentID and c.createdBy=<pid>

TODO:  what is the best way to populate the recipients?

 

Sorting + pagination

Paginated results<Message>Gets all messages the authenticated user has sentAuthenticated User
  •  POST
/messagen/aMessageinReplyTo=null (i.e. another Message)MessageSends a message.  Note, message delivery permission is on a recipient-by-recipient basis, asynchronous to the message creation.  Unauthorized delivery may result in silent failure or a bounce message (TBD).

Authenticated User

 

Must be admin to send to AUTH_USERS

  •  POST
/message/checkuse existing userGroupDAO query to check recipients.MessageinReplyTo=null (i.e. another Message)

Boolean

ErrorResponse (question)

Checks to see if the message is correctly formatted and that the sender has permission to send to the indicated recipientsAuthenticated User
  •  POST
/message/{id}/forwardn/aRecipientBundle  Forwards a message to other recipients. This is equivalent to getting a (visible) message and POST-ing it to /messageSender or Receiver
  •  POST
/entity/{id}/commentsn/aMessage MessageConvenience method for commenting on an entity. The service fills out fields like message ID, thread ID, and recipients, leaving just subject and body for the user.

Authenticated user with SEND_MESSAGE permission on entity

  •  GET
/message/{id}select * from messagecontent c, messagetouser m where c.ID=m.messageContentID and c.ID=<id>  MessageGets a specific messageSender or Receiver
  •  GET
/message/{id}/conversationTODO 

Sorting + pagination

Paginated results<Message>Gets messages belonging in the same thread as the message ID.  The list is filtered according to the user's ID.Sender or Receiver
  •  GET
/entity/{id}/comments

select * from messagecontent mc, comment c, inreploytoroot r

where

mc.ID=c.messageContentID and c.TargetType='ENTITY" and c.TargetID=<id>

 

Sorting + pagination

Paginated results<Message>Gets message belonging to the thread tied to the entity.Authenticated user with READ permission on entity
  •  GET
/message/{id}/statusselect * from messagestatus where messagstatus.messageContentID=<id>  Message StatusGets the status of a messageReceiver
  •  PUT
/message/statusn/aMessageStatus  

Marks a message as:

  • READ
  • UNREAD
  • ARCHIVED
Receiver
  •  GET
/message/settingsTODO  MessageSettingsGets the notification settings of the userAuthenticated User
  •  PUT
/message/settingsTODOMessageSettings  Changes notification settingsAuthenticated User

'* 'replyTo' is a misnomer.  In email 'replyTo' is the address of the person to whom you send an email reply.  In this case it's the ID of the message referenced by the current message.  So it should be called 'inReplyTo' or some other distinctive name.

...