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

Disable message logs upon using private otr chats. #49

Open
redragonx opened this issue Jun 14, 2015 · 5 comments
Open

Disable message logs upon using private otr chats. #49

redragonx opened this issue Jun 14, 2015 · 5 comments

Comments

Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
@redragonx
Copy link

@redragonx redragonx commented Jun 14, 2015

Pidgin-otr does this by default.

@arlolra
Copy link
Owner

@arlolra arlolra commented Jun 15, 2015

Ah, right. In Tor Messenger we've disabled all logging. But for Instantbird, this will be a useful feature.

@azadi
Copy link
Collaborator

@azadi azadi commented Jun 16, 2015

Related: warn the user if logging is enabled (if the user explicitly enabled it).

arlolra added a commit that referenced this issue Jun 17, 2015
 * Logging is disabled by default in Tor Messenger but useful for
   Instantbird users.

 * Issue #49
@arlolra
Copy link
Owner

@arlolra arlolra commented Jun 17, 2015

Pushed a patch for the first part of this there. It depends on merging something upstream though.
https://bugzilla.mozilla.org/show_bug.cgi?id=1175374

Working on the warning next.

@vqhuy
Copy link
Collaborator

@vqhuy vqhuy commented Mar 10, 2016

@arlolra how do you display the warning to users? Should we show it in conversation window (using systemMessage method)?

@arlolra
Copy link
Owner

@arlolra arlolra commented Mar 10, 2016

@c633 I think using the notification box would be preferable, since we'll eventually want to conveniently add actions like "disable logging".

Unfortunately, at present, disabling logging doesn't affect the current conversation. I had a patch for that but it got stalled on a dependency, which I see has now resolved. I should pick that up soon.
https://bugzilla.mozilla.org/show_bug.cgi?id=1175706

Even more unfortunate is that the logging flag has no granularity and can't be applied to individual conversations. The other patch above, which also stalled, is perhaps too fined grained and applies to messages themselves. Not sure why I did that. I need to revisit this whole issue.

Anyways, yes, a notification box with perhaps a pref to permanently dismiss the warnings would be a good first start. See code in ui.js around notifyBox for an example.

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