Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Recent messages" for locked lists is available by URL #14

Open
jjfine opened this issue Oct 17, 2013 · 4 comments
Open

"Recent messages" for locked lists is available by URL #14

jjfine opened this issue Oct 17, 2013 · 4 comments

Comments

@jjfine
Copy link

jjfine commented Oct 17, 2013

Issue by Trevoke from Sunday Jan 06, 2013 at 00:29 GMT
Originally opened as https://github.com/cyrusinnovation/ListList/issues/38


I was able to see the recent messages for a locked list of which I am not a member by typing in the URL for the Recent Messages for that list manually -- e.g.

http://cyruslists.com/lists/99/archives?

This should, of course, not be possible.

@jjfine
Copy link
Author

jjfine commented Oct 17, 2013

Comment by pinfieldharm from Sunday Jan 06, 2013 at 01:10 GMT


Thanks for finding this. Until this is fixed, I nuked the archives in the Rails cache, changed the password for the Gmail firehose account, and deleted all the messages therein.

@jjfine
Copy link
Author

jjfine commented Oct 17, 2013

Comment by pinfieldharm from Sunday Jan 06, 2013 at 01:11 GMT


Messages to closed lists should absolutely, positively not be going into the firehose gmail account.

@jjfine
Copy link
Author

jjfine commented Oct 17, 2013

Comment by Trevoke from Sunday Jan 06, 2013 at 01:13 GMT


So for now we're not worrying about changing the locked status of a list. However, if we -could- change the status of a list, we'd have to manage :

  • closed -> open means we begin to store
  • open -> close means we clear the cache

Correct? Because, well, correct me if I'm wrong, we can still change that status through the database.. Is this a use case we want to worry about?

@jjfine
Copy link
Author

jjfine commented Oct 17, 2013

Comment by pinfieldharm from Sunday Jan 06, 2013 at 01:19 GMT


I wouldn't worry about retroactively clearing the firehose account. I think the contract we are enforcing is "mail sent to a closed list won't be archived." If a list is open when a message is sent, then it's okay to keep it forever.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants