Excel Online team also believes COI requirement is not needed for this API.
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 207
- Forks
- 17
- PR merge metrics
- No merged PRs in 30d
Description
Excel Online team also believes COI requirement is not needed for this API.
After deep discussions on that decision we came to understanding that the API does not expose any new vulnerability to time channel attack as the timings from the API output is the same as perfromance.now(after the clamping obviously).
Further more the sampling interval of the profiler is in millisecond resolution as well, so again nothing new is exposed here.
@acomminos, if you agree on the timing concern, can we update this part(https://wicg.github.io/js-self-profiling/#privacy-security) in the spec to reflect our agreement on this understanding.
Another aspect when requiring COI is avoiding leaking information form cross-origin scripts.
However this is already being handled inside the API by avoiding function names introspection if the script is from different origin and did not provided CORS header, just as in error.callstack AFAIU.
Therefore we are clear here as well.
Taking those points into account we strongly believe that the COI in this case is redundant.
Thanks.
Originally posted by @magenish in https://github.com/WICG/js-self-profiling/issues/41#issuecomment-846522442
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Read the privacy-security section of the linked js-self-profiling specification and review the referenced discussion in issue 41, including the handling described for error.callstack. Determine whether the COI requirement should be removed or revised based on the timing and cross-origin arguments; done means the specification reflects an agreed decision.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- api, documentation, security
- Issue type
- Documentation
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100