Repository navigation
Code to check quota on backend #911
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
Merged
Merged
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
76e9d08
Some basic infrastructure
kjetilk a162ef4
Reading from file works
kjetilk c4e59cb
Test for non-existant file
kjetilk 423297b
Set no quota if the predicate isn't there
kjetilk fb35d01
Introduce get-folder-size dependency
kjetilk b3298b5
Not-yet working code for over quota
kjetilk 0f9c04d
turn the signature
kjetilk 38c023e
Improve the callback async handling, thanks @megoth
kjetilk 97d81eb
Make the quota functions async
kjetilk 2b6a272
Still failing async tests
kjetilk 0b02130
Fix the tests to async, thanks @megoth
kjetilk 889ed4a
Comment on the implementation
kjetilk 2ebd8ad
Add a negative test
kjetilk fe9581c
Make actualSize function part of the public API
kjetilk 2cc260c
Improve documentation somewhat
kjetilk 3d30cc4
add cache TODO
kjetilk 22b21f7
Some basic infrastructure
kjetilk 58a33e4
Reading from file works
kjetilk 3383003
Test for non-existant file
kjetilk 383b849
Set no quota if the predicate isn't there
kjetilk a5c968f
Introduce get-folder-size dependency
kjetilk 2ec4905
Not-yet working code for over quota
kjetilk 53fe1b1
turn the signature
kjetilk 2c51be3
Improve the callback async handling, thanks @megoth
kjetilk 7dbaa1e
Make the quota functions async
kjetilk 4d09c2e
Still failing async tests
kjetilk 790e8ea
Fix the tests to async, thanks @megoth
kjetilk 1a54eb8
Comment on the implementation
kjetilk 50cb63f
Add a negative test
kjetilk 8b42908
Make actualSize function part of the public API
kjetilk 43be6a1
Improve documentation somewhat
kjetilk 34aa3bf
add cache TODO
kjetilk 4925cef
Merge branch 'feature/check-quota' of github.com:solid/node-solid-ser…
kjetilk 6378213
Failing LDP tests
kjetilk 1b38a86
Must use callback
megoth 8f082e9
Comment out my own tests
kjetilk 0eec430
Check quota in PATCH too
kjetilk a5516b1
more docs
kjetilk c5082fa
improve wording
kjetilk a99a0ee
rename other to fs for debugging
kjetilk 9f171d7
improve test description
kjetilk 7423905
skip failing tests
kjetilk 2327cd1
skip another test
kjetilk e981aa5
Merge branch 'master' into feature/check-quota
kjetilk 7b2f773
Remove byte units and the logic to support it. Also use anyValue method
kjetilk 3dcddce
Remove a console.log
kjetilk 4deae1e
Merge branch 'master' into feature/check-quota
kjetilk a5a9617
Update package.json
kjetilk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| @prefix dct: <http://purl.org/dc/terms/>. | ||
| @prefix pim: <http://www.w3.org/ns/pim/space#>. | ||
| @prefix solid: <http://www.w3.org/ns/solid/terms#>. | ||
| @prefix unit: <http://www.w3.invalid/ns#>. | ||
|
|
||
| <> | ||
| a pim:ConfigurationFile; | ||
|
|
||
| dct:description "Administrative settings for the POD that the user can only read." . | ||
|
|
||
| </> | ||
| solid:storageQuota "25000000" . | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| @prefix acl: <http://www.w3.org/ns/auth/acl#>. | ||
| @prefix foaf: <http://xmlns.com/foaf/0.1/>. | ||
|
|
||
| <#owner> | ||
| a acl:Authorization; | ||
|
|
||
| acl:agent | ||
| <{{webId}}>; | ||
|
|
||
| acl:accessTo <./serverSide.ttl>; | ||
|
|
||
| acl:mode acl:Read . | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note for vNext: we should probably be able to implement this with a decorator pattern only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good. For example with traits?