Skip to content
This repository was archived by the owner on Dec 18, 2018. It is now read-only.

Don't log message content in the JavaScript client #1449

Closed
analogrelay opened this issue Feb 15, 2018 · 0 comments
Closed

Don't log message content in the JavaScript client #1449

analogrelay opened this issue Feb 15, 2018 · 0 comments
Assignees
@JamesNK
Labels
3 - Done cost: S Will take up to 2 days to complete type: Bug
Milestone
2.1.0-preview2

Comments

@analogrelay
Copy link
Contributor

We generally avoid logging things that may have sensitive data in them, we should avoid logging the message content in the JavaScript client. I'd certainly expect users to ship logs from the client over to a telemetry service, so they should be treated just as sensitively as server-side logs.
@analogrelay analogrelay mentioned this issue Feb 15, 2018
Closed
@muratg muratg added this to the 2.1.0-preview2 milestone Feb 20, 2018
@mikaelm12 mikaelm12 mentioned this issue Feb 25, 2018
Closed
@analogrelay analogrelay added the cost: S Will take up to 2 days to complete label Mar 16, 2018
@analogrelay analogrelay modified the milestones: 2.1.0-preview2, 2.1.0-rc1 Mar 16, 2018
@analogrelay analogrelay modified the milestones: 2.1.0-rc1, 2.1.0-preview2 Mar 22, 2018
@analogrelay analogrelay assigned JamesNK and unassigned JamesNK Mar 22, 2018
@JamesNK JamesNK self-assigned this Mar 22, 2018
@JamesNK JamesNK mentioned this issue Mar 23, 2018
Merged
@JamesNK JamesNK closed this as completed Mar 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@JamesNK JamesNK

Labels
3 - Done cost: S Will take up to 2 days to complete type: Bug
Projects
None yet
Milestone
2.1.0-preview2
Development

No branches or pull requests
5 participants
@analogrelay @davidfowl @JamesNK @muratg @mikaelm12