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

Security considerations #72

Open
Relequestual opened this issue Nov 21, 2018 · 4 comments
Open

Security considerations #72

Relequestual opened this issue Nov 21, 2018 · 4 comments
Assignees
Milestone

Comments

@Relequestual
Copy link
Member

As mentioned in the RC5 Retrospective document:

While IETF RFC documents often do not extensivly mention general web based security considerations, and only those relating specifically to the document at hand, in this instance it could be beneficial to mention a number of additional security considerations for benefit of this specifications audience.

We should note security considerations:

  • Data from remote sources should be treated as untrusted
  • Must use HTTPS - Verify certificates
  • Use standard JSON deserialisation libraries
  • Limit searching queries so as to prevent accidental Denial of Service attacks
  • Follow Cross Site Scripting (XSS) prevention rules as defined by OWASP XSS Prevention Cheat Sheet with a brief overview of rules 0, 1, 2, and 3. This is for the client side
  • Follow SQL Injection Prevention guide from OWASP. This is for the server and client side
  • Note that the two OWASP guides fall under "good practice"
  • rfc8259 The JavaScript Object Notation (JSON) Data Interchange Format - AKA the JSON specification, highlights a security consideration that is applicable
@Relequestual Relequestual self-assigned this Nov 21, 2018
@Relequestual Relequestual added this to the 1.0.0 RC6 milestone Dec 3, 2018
@Relequestual
Copy link
Member Author

@kemp-google Did you say you're willing to take this on? If so, I'll re-assign to you! =]

@kemp-google
Copy link

Thanks @Relequestual. Did you have a desired location for this content? If not I imagine a "Security Considerations" section in specification.md.

@rishidev
Copy link

The security considerations have traditionally gone into the main specification. Also htsget has some standard GA4GH language on the Cross Origin Resource Sharing that may be useful

@Relequestual
Copy link
Member Author

A new security document has been created by the Security worksteam.
I have filled most of this in: https://drive.google.com/open?id=1IIqzk6wrphqXHNN72BKZA9nvRm0_efPwEbTO8pYTcRc
I was unclear if all of the form was relevant to this specification.
Rish said the document should be submitted as is to open a discussion.

@rishidev Can you let us know when we should expect to hear back on the submission?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants