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

Decision Proposal 090 - Banking Maintenance Iteration 1 #90

Closed
CDR-API-Stream opened this issue Nov 24, 2019 · 2 comments
Closed

Decision Proposal 090 - Banking Maintenance Iteration 1 #90

CDR-API-Stream opened this issue Nov 24, 2019 · 2 comments
Assignees
Labels
Category: API A proposal for a decision to be made for the API Standards made Category: InfoSec Information Security Technical Working Group Decision Proposal Status: Decision Made A determination on this decision has been made

Comments

@CDR-API-Stream
Copy link
Contributor

CDR-API-Stream commented Nov 24, 2019

This decision captures the outcome of Banking Maintenance Iteration 1. Consultation on these items was conducted on the standards maintenance repository.

This decision will cover the following items:

The final decision document has been reviewed and approved by the Data Standards Chair. It is attached below:
Decision 090 - Banking Maintenance Iteration 1.pdf

@CDR-API-Stream CDR-API-Stream added Category: API A proposal for a decision to be made for the API Standards made Status: Proposal Pending A proposal for the decision is still pending Category: InfoSec Information Security Technical Working Group Decision Proposal labels Nov 24, 2019
@CDR-API-Stream CDR-API-Stream self-assigned this Nov 24, 2019
@CDR-API-Stream CDR-API-Stream added Status: Decision Made A determination on this decision has been made and removed Status: Proposal Pending A proposal for the decision is still pending labels Dec 3, 2019
@perlboy
Copy link
Contributor

perlboy commented Dec 6, 2019

Adoption of CORS as per FAPI standard

I repeat again that the FAPI Standard DOES NOT mandate CORS. It is an optional requirement at the discretion of the Holder/OP. This subject should be reworded to highlight that the CDS has made a deliberate decision to convert a MAY to a MUST.

Specifically quoting from the only part of the FAPI spec that mentions CORS in 6.2.1. Protected resources provisions item 13:

should support the use of Cross Origin Resource Sharing (CORS) [CORS] and or other methods as appropriate to enable JavaScript clients to access the endpoint if it decides to provide access to JavaScript clients.
NOTE: Providing access to JavaScript clients has other security implications. Before supporting those clients [RFC6819] should be consulted.

In addition, the CDS proposes this change for NON Protected resources only which is a further divergence from the clauses referred to in the FAPI specification.

A more suitable subject would be Introduction of CORS to Unauthenticated Endpoints coupled with a note that the FAPI spec provides allowances but this is a profile specific modification being adopted by the DSB.

Further, a note should be made that the Independent InfoSec Review conducted specifically stated the threat assessment was based on CORS headers being absent and consequently "has no negative security impact, but may need to be specified in future if browser-side access is required, at which point cross-domain security attacks can emerge".

@CDR-API-Stream CDR-API-Stream added the Status: Feedback Period Closed The feedback period is complete and a final decision is being formulated label Dec 10, 2019
@ConsumerDataStandardsAustralia ConsumerDataStandardsAustralia locked and limited conversation to collaborators Jan 13, 2020
@CDR-API-Stream CDR-API-Stream removed the Status: Feedback Period Closed The feedback period is complete and a final decision is being formulated label Jul 31, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Category: API A proposal for a decision to be made for the API Standards made Category: InfoSec Information Security Technical Working Group Decision Proposal Status: Decision Made A determination on this decision has been made
Projects
None yet
Development

No branches or pull requests

2 participants