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

Update SPARQL DDoS ed note #107

Merged
merged 3 commits into from
Feb 1, 2021
Merged

Conversation

mmccool
Copy link
Contributor

@mmccool mmccool commented Jan 4, 2021

Tweak ed note; please check


Preview | Diff

@AndreaCimminoArriaga
Copy link
Contributor

I agree with the note, except for this part "or complexity of queries". I think that the complexity of a query should not be considered as a security thread. Maybe can we remove this part?

@mmccool
Copy link
Contributor Author

mmccool commented Jan 25, 2021

Complexity of queries is a security threat because it can be used for a DDoS attack, e.g. submitting a query that takes a pathological amount of time to complete to tie up the server. This is also applicable to JSONpath to some extent. So it applies to all query types, not just SPARQL, so we can cover it under "Security and Privacy Considerations" where mitigations can be mentioned (e.g. watchdog timers).

@mmccool
Copy link
Contributor Author

mmccool commented Jan 25, 2021

To Do: move DDoS discussion and mitigation to "Security and Privacy Considerations". See also w3c/wot-security#196 which will be discussed in the WoT Security meeting, so it would be good to mention possible security concerns there.

move DDoS to security and privacy consideration section
add header to fix rendering
@mmccool
Copy link
Contributor Author

mmccool commented Jan 26, 2021

Updated to move DDoS discussion to S&PC section. Other considerations should be added later.

@mmccool mmccool merged commit 0db5641 into master Feb 1, 2021
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

Successfully merging this pull request may close these issues.

2 participants