Skip to content

[PB-6473]: feat/gateway-user-usage#223

Open
jzunigax2 wants to merge 2 commits into
masterfrom
feat/gateway-user-usage
Open

[PB-6473]: feat/gateway-user-usage#223
jzunigax2 wants to merge 2 commits into
masterfrom
feat/gateway-user-usage

Conversation

@jzunigax2

Copy link
Copy Markdown

Implemented a new endpoint to retrieve user storage usage, returning a snapshot of max and total used space. Added corresponding interface and unit tests to ensure functionality. Updated HTTP gateway routes to include the new endpoint.

Implemented a new endpoint to retrieve user storage usage, returning a snapshot of max and total used space. Added corresponding interface and unit tests to ensure functionality. Updated HTTP gateway routes to include the new endpoint.
Comment thread lib/server/http/gateway/controller.ts Outdated
) {
const { uuid } = req.params;

if (!uuid) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Validate the format also, as a good practise

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

right, done

})
})

describe('Getting user usage', () => {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a 401 test always, as the gateway controller is especially delicate in terms of security

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addressed

@jzunigax2 jzunigax2 requested a review from sg-gs July 6, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants