FAPI 1.0 Final Preview Feedback

Issue #368 resolved
Edmund Jay created an issue

Brian Campbell:

In baseline:
"This document is Part 1 of FAPI Security Profile 1.0 that specifies the Financial-grade API and it provides a profile of OAuth that is suitable to be used in the access of read-only financial data and similar use cases. A higher level of security profile is provided in Part 2, suitable for read and write financial access APIs and other similar situations where the risk is higher. " is maybe okay because of the "and similar..." qualifications
but
"obtain OAuth tokens in a secure manner for read-only access to protected data"
"use tokens to read protected data from REST endpoints."
"Read-only access is generally viewed to pose a lower risk than the write access and as such, the characteristics required of the tokens are different and the methods to obtain tokens are explained separately."
"Read-only access is a lower risk scenario compared to the write access; therefore the protection level can also be lower."
"shall verify that the scope associated with the access token authorizes the reading of the resource it is representing"
don't really make sense in the context of a document that isn't supposed to be about read-only
also the grammar check in my email doesn't like "to the write"

In advanced:
"provides a profile of OAuth that is suitable to be used for high risk access (read or write), for example, read access to highly sensitive data or write access to financial data (also known as payment initiation)." and "For example, read and write access to a bank API has a higher financial risk than read-only access." are maybe ok because they are given as examples rather than absolutes
but
"Read and write access carries higher risk; therefore the protection level required is higher than read-only access."
looks like it just wasn't updated with the change from read and write to advanced

Comments (2)

  1. Edmund Jay reporter

    Merged in issue_368 (pull request #231)

    fixes #368 - rewrite text regarding read/write

    • fixes #368 - rewrite text regarding read/write

    • fixes #369 - Update FAPI 1.0 contributors list

    • Updates #368 - feedback for read/write text

    • updates #368 - additional feedback changes

    Approved-by: Dima Postnikov Approved-by: Dave Tonge Approved-by: Joseph Heenan Approved-by: Stuart Low Approved-by: Ralph Bragg

    → <<cset f70b08770dbe>>

  2. Edmund Jay reporter

    Merged in issue_368 (pull request #231)

    fixes #368 - rewrite text regarding read/write

    • fixes #368 - rewrite text regarding read/write

    • fixes #369 - Update FAPI 1.0 contributors list

    • Updates #368 - feedback for read/write text

    • updates #368 - additional feedback changes

    Approved-by: Dima Postnikov Approved-by: Dave Tonge Approved-by: Joseph Heenan Approved-by: Stuart Low Approved-by: Ralph Bragg

    → <<cset f70b08770dbe>>

  3. Log in to comment